Skip to content
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

fix(web-components): ensure buttons with long text maintain center alignment #31179

Conversation

chrisdholt
Copy link
Member

Previous Behavior

Behavior regressed for buttons with long text.

New Behavior

This PR ensures that buttons with long text have the text center aligned when wrapping.

Related Issue(s)

  • Fixes #

@chrisdholt chrisdholt requested a review from a team as a code owner April 24, 2024 15:45
@chrisdholt chrisdholt force-pushed the users/chhol/fix-button-vr-tests branch from b500d93 to 6f97f26 Compare April 24, 2024 15:48
@chrisdholt chrisdholt changed the title fix(web-components): ensure ebuttons with long text maintain center alignment fix(web-components): ensure buttons with long text maintain center alignment Apr 24, 2024
@fabricteam
Copy link
Collaborator

📊 Bundle size report

🤖 This report was generated against b8e0cea8051975886a2d8553bf2771d85a5dd283

@@ -0,0 +1,7 @@
{
Copy link
Collaborator

Choose a reason for hiding this comment

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

🕵 fluentui-web-components-v3 Open the Visual Regressions report to inspect the affected screenshots

Text 1 screenshots
Image Name Diff(in Pixels) Image Type
Text.Block.normal.chromium.png 995 Changed

@chrisdholt chrisdholt merged commit ee42c7c into microsoft:web-components-v3 Apr 24, 2024
16 of 17 checks passed
@chrisdholt chrisdholt deleted the users/chhol/fix-button-vr-tests branch April 24, 2024 15:57
Copy link

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

chrisdholt added a commit that referenced this pull request Apr 29, 2024
radium-v pushed a commit to radium-v/fluentui that referenced this pull request Apr 29, 2024
radium-v pushed a commit to radium-v/fluentui that referenced this pull request Apr 29, 2024
radium-v pushed a commit to radium-v/fluentui that referenced this pull request Apr 30, 2024
radium-v pushed a commit that referenced this pull request Apr 30, 2024
radium-v pushed a commit that referenced this pull request May 2, 2024
radium-v pushed a commit that referenced this pull request May 2, 2024
radium-v pushed a commit that referenced this pull request May 2, 2024
radium-v pushed a commit that referenced this pull request May 3, 2024
radium-v pushed a commit that referenced this pull request May 6, 2024
radium-v pushed a commit that referenced this pull request May 6, 2024
radium-v pushed a commit that referenced this pull request May 8, 2024
radium-v pushed a commit that referenced this pull request May 10, 2024
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