-
Notifications
You must be signed in to change notification settings - Fork 33
Add Button directive #2372
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Button directive #2372
Conversation
🔍 Preview links for changed docs |
Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
…to button-directive
reakaleek
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Do you have any thoughts on this @itsalexcm?
|
@itsalexcm Thanks! I would guess it's for conveying a secondary or optional action, like we do in the homepage of our docs. |
|
@theletterf Fair enough. I think that in the product we usually use the outlined button to highlight or frame actions (e.g. headers, marketing contexts), while standard secondary actions typically use the defined non-filled primary button style. That said, I’m fine with the outlined button for now. We can revisit and align this later if needed. Good job! |

Adds a new
{button}directive for creating styled call-to-action buttons in documentation.Fixes #2373
Features
primary(filled) andsecondary(outlined){button-group}left,center,rightfor standalone buttons and groupstarget="_blank"and security attributesSyntax
:::{button} Get Started :link: /get-started :type: primary ::: ::::{button-group} :::{button} Primary :link: docs-content://get-started/introduction.md ::: :::{button} Secondary :link: /other-page :type: secondary ::: ::::Changes
ButtonBlock,ButtonGroupBlockdocs/syntax/buttons.mdNote
This PR was developed with AI assistance (Claude Opus 4.5 in Cursor 2).