Skip to content

fix(button): update styling sample Sass functions#3780

Merged
simeonoff merged 3 commits into
masterfrom
apetrov/update-button-style-sample
Nov 21, 2025
Merged

fix(button): update styling sample Sass functions#3780
simeonoff merged 3 commits into
masterfrom
apetrov/update-button-style-sample

Conversation

@adrianptrv

@adrianptrv adrianptrv commented Nov 14, 2025

Copy link
Copy Markdown
Contributor

Closes #6252

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 updates the Sass styling sample for button components by replacing the deprecated button-theme function with more specific theme functions (contained-button-theme, outlined-button-theme, and flat-button-theme). The changes align with updated API design patterns that provide separate theming functions for different button variants.

Key Changes

  • Replaced single button-theme function with variant-specific theme functions
  • Updated theme property names to match the new API (removed some border-related properties, added new focus/active properties)
  • Added separate theme examples for contained, outlined, and flat button variants

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

Comment thread src/app/data-entries/buttons/buttons-style/buttons-style.component.scss Outdated
adrianptrv and others added 2 commits November 14, 2025 18:50
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@simeonoff simeonoff merged commit b2d7209 into master Nov 21, 2025
4 checks passed
@simeonoff simeonoff deleted the apetrov/update-button-style-sample branch November 21, 2025 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update Button styling section

3 participants