dequelabs / dequelabs/cauldron

Sync TopBarItem/TopBarTrigger with UXPin

Open
#1,211 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
TypeScript
Stars
127
Forks
31
Avg merge
2d 12h
Merged PRs (30d)
8

Description

## Things to Change

- No more bottom border on hover, just a background color change
- Focus state should use the default focus ring instead of bottom border
- Bottom border should only be visible when trigger is active (this could be set via an `active` prop or checking for `aria-current`)
- Font-weight for both Item/Trigger should match (`400`)

Other considerations:

MenuBar should always be aligned to the right when inside of the TopBar component.

See [design system](https://app.uxpin.com/design-system/c933c74928e2195cc540/ui-patterns/213721) for examples of what values to use for the various states. Reach out to @dequelabs/cauldron-design if anything is unclear.

Contributor guide

Open the contributing guide

Research direction

Locate the TopBarItem, TopBarTrigger, and MenuBar components, then compare their current hover, focus, active, font-weight, and alignment behavior with the linked UXPin design system. Done means the requested states and right alignment match the design examples; contact @dequelabs/cauldron-design if any values remain unclear.

Written by the indexing model from the issue text.

Assessment

Tech stack
react, typescript
Domain
design, frontend
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.