Skip to content

refactor(fortal): consolidate base button state styling - #150

Merged
leoafarias merged 2 commits into
mainfrom
refactor/button-base
Aug 13, 2026
Merged

refactor(fortal): consolidate base button state styling#150
leoafarias merged 2 commits into
mainfrom
refactor/button-base

Conversation

@leoafarias

Copy link
Copy Markdown
Member

Description

Consolidates duplicated Fortal Button and IconButton visual-state definitions behind their shared Radix BaseButton boundary while retaining each component's geometry and styler adapter. It also replaces the Checkbox, Radio, and TextField effect-layer pass-through helpers with direct RemixBoxEffectLayerMix construction. Characterization coverage now compares every BaseButton variant across contrast and interaction states, including focus precedence and classic pressed offsets; package analysis and all 378 remix_fortal tests pass.

Related Issues

None.


Checklist

Note: Updating the pubspec.yaml and CHANGELOG.md is not required. These are handled automatically during the release process.

  • My PR includes unit or integration tests for all changed/updated/fixed behaviors.
  • I have updated or added relevant documentation (doc comments with ///).
  • I am prepared to follow up on review comments in a timely manner.

Breaking Change

Does this PR require users of the package to manually update their code?

  • Yes, this is a breaking change.
  • No, this is not a breaking change.

@leoafarias
leoafarias merged commit a4ca6cb into main Aug 13, 2026
3 checks passed
@leoafarias
leoafarias deleted the refactor/button-base branch August 13, 2026 22:08
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.

1 participant