Skip to content

fix(button): use correct border radius token on plain buttons#8175

Merged
mcoker merged 1 commit into
patternfly:mainfrom
srambach:bugfix-plain-button-radius
Feb 26, 2026
Merged

fix(button): use correct border radius token on plain buttons#8175
mcoker merged 1 commit into
patternfly:mainfrom
srambach:bugfix-plain-button-radius

Conversation

@srambach

@srambach srambach commented Feb 26, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Member

Missed using a new token for the plain button border radius.

Summary by CodeRabbit

  • Style
    • Updated Plain button variant styling.

@coderabbitai

coderabbitai Bot commented Feb 26, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Repository UI (base), Organization UI (inherited)

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e7c7d97 and be5b019.

📒 Files selected for processing (1)
  • src/patternfly/components/Button/button.scss

Walkthrough

Modified the Plain button variant's border-radius property from small to action--plain--default in the button component's SCSS file. No functional changes or control flow modifications.

Changes

Cohort / File(s) Summary
Button Styling
src/patternfly/components/Button/button.scss		 Updated Plain button variant border-radius from small to action--plain--default.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • lboehling
  • kmcfaul
  • mcoker
🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title follows conventional commit format with type 'fix' and scope 'button', and accurately describes the main change: updating the border radius token for plain buttons.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@patternfly-build

patternfly-build commented Feb 26, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Preview: https://pf-pr-8175.surge.sh

A11y report: https://pf-pr-8175-a11y.surge.sh

@mcoker mcoker merged commit 744794d into patternfly:main Feb 26, 2026
5 checks passed
@patternfly-build

patternfly-build commented Feb 26, 2026

Copy link
Copy Markdown
Collaborator

🎉 This PR is included in version 6.5.0-prerelease.45 🎉

The release is available on:

Your semantic-release bot 📦🚀

This was referenced Mar 2, 2026
Closed
Closed
@coderabbitai coderabbitai Bot mentioned this pull request Mar 11, 2026
Merged
@coderabbitai coderabbitai Bot mentioned this pull request Apr 22, 2026
Merged
@coderabbitai coderabbitai Bot mentioned this pull request May 15, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mcoker mcoker mcoker approved these changes

@kmcfaul kmcfaul Awaiting requested review from kmcfaul

@lboehling lboehling Awaiting requested review from lboehling

Assignees

No one assigned

Labels

released on @prerelease

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@srambach @patternfly-build @mcoker