Skip to content

fix(button): remove border width animation - #7832

Merged
mcoker merged 1 commit into
patternfly:mainfrom
srambach:7743-button-border-animation-hi-c
Sep 15, 2025
Merged

mcoker merged 1 commit into
patternfly:mainfrom
srambach:7743-button-border-animation-hi-c

Conversation

@srambach

Copy link
Copy Markdown
Member

Fixes #7743

After experimentation, it seems that the perceived jankiness in the plain button hover animation was because we are animating in a 1px border. Because it's simply going from 0 to 1px, there is very room for intermediate states and so it seems jumpy. I've removed border-width from the transition properties. Now the plain button (and other) button borders will animate in based just on the color, but this seems to have a smoother feel.

@patternfly-build

patternfly-build commented Sep 15, 2025 •

Copy link
Copy Markdown
Collaborator

@mcoker mcoker 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.

L🤖TM!

@mcoker
mcoker requested a review from lboehling September 15, 2025 18:13

@lboehling lboehling left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

this feels so much smoother to me across the board -- always felt like the border transition felt a little jumpy!

@mcoker
mcoker merged commit a18eafa into patternfly:main Sep 15, 2025
4 checks passed
@patternfly-build

Copy link
Copy Markdown
Collaborator

🎉 This PR is included in version 6.3.0-prerelease.71 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

Button - high contrast animation performance

4 participants