Skip to content

feat(actions): unify page action buttons#7216

Merged
sanderson merged 4 commits into
masterfrom
ux-page-actions
May 14, 2026
Merged

feat(actions): unify page action buttons#7216
sanderson merged 4 commits into
masterfrom
ux-page-actions

Conversation

@sanderson
Copy link
Copy Markdown
Collaborator

This PR sets out to accomplish the following:

  • Simplify the Telegraf plugin md generation by automatically generating links to source code
  • Provide a download mechanism for downloading files from an external source (used for downloading sample.conf files for each plugin.
  • Unify the styles of the added Telegraf actions and the existing LLM format selector:
image

Checklist

  • Rebased/mergeable
  • Local build passes (npx hugo --quiet)

@sanderson sanderson requested a review from a team as a code owner May 13, 2026 23:37
@sanderson sanderson requested review from jstirnaman and removed request for a team May 13, 2026 23:37
Copy link
Copy Markdown
Contributor

@jstirnaman jstirnaman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! 1 suggestion to improve DX. Also, any suggestions/documentation on when we should or shouldn't use the page-actions components would be helpful.

Comment thread layouts/partials/article/telegraf-plugin-links.html Outdated
@sanderson
Copy link
Copy Markdown
Collaborator Author

Also, any suggestions/documentation on when we should or shouldn't use the page-actions components would be helpful.

I'll add docs for this, but in my mind, these should be used for actions that are general, but still specific to the current page. For example, in the Flux stdlib, we could use these to link to package source code. Or in the InfluxDB 3 plugin library, we could link to the plugin source. Etc.

These shouldn't really be used to for a single piece of functionality on a single page.

@github-actions
Copy link
Copy Markdown
Contributor

Vale Style Check Results

Metric Count
Errors 0
Warnings 7
Suggestions 18
Warnings (7)
File Line Rule Message
.github/instructions/layouts.instructions.md 55 write-good.TooWordy 'Implement' is too wordy.
DOCS-PAGE-ACTIONS.md 7 write-good.TooWordy 'it is' is too wordy.
DOCS-PAGE-ACTIONS.md 28 write-good.TooWordy 'all of' is too wordy.
DOCS-PAGE-ACTIONS.md 35 write-good.Passive 'is derived' may be passive voice. Use active voice if you can.
DOCS-PAGE-ACTIONS.md 64 write-good.Weasel 'usually' is a weasel word!
DOCS-PAGE-ACTIONS.md 87 write-good.Weasel 'usually' is a weasel word!
DOCS-PAGE-ACTIONS.md 90 write-good.Weasel 'usually' is a weasel word!

Check passed

@sanderson
Copy link
Copy Markdown
Collaborator Author

@sanderson sanderson merged commit ac3fc72 into master May 14, 2026
22 checks passed
@sanderson sanderson deleted the ux-page-actions branch May 14, 2026 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants