Skip to content
This repository was archived by the owner on Jun 8, 2026. It is now read-only.

docs(button): fix paragraph position and URLs - #6467

Merged
simeonoff merged 3 commits into
vnextfrom
apetrov/fix-button-styling-text
May 11, 2026
Merged

docs(button): fix paragraph position and URLs#6467
simeonoff merged 3 commits into
vnextfrom
apetrov/fix-button-styling-text

Conversation

@adrianptrv

Copy link
Copy Markdown
Contributor

Move Sass styling paragraph to correct section and fix broken URLs.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR adjusts the Button component documentation by relocating a Sass-styling paragraph to a more appropriate section and updating related links.

Changes:

  • Moved the Sass styling paragraph from the theme tables area to the Sass styling section.
  • Updated/standardized the Sass API links for the theme functions.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread en/components/button.md Outdated
Comment on lines +1407 to +1410
Another way to style the button is by using **Sass**, along with our type-specific theme functions: [`flat-button-theme`]({environment:sassApiUrl}/themes#function-flat-button-theme), [`outlined-button-theme`]({environment:sassApiUrl}/themes#function-outlined-button-theme), [`contained-button-theme`]({environment:sassApiUrl}/themes#function-contained-button-theme), and [`fab-button-theme`]({environment:sassApiUrl}/themes#function-fab-button-theme).

Each of them will target only the buttons of that specific type.

@simeonoff
simeonoff merged commit d71dd5b into vnext May 11, 2026
5 checks passed
@simeonoff
simeonoff deleted the apetrov/fix-button-styling-text branch May 11, 2026 07:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants