Border radius mismatches between chrome blocks and outer window
Categories
(Firefox :: Theme, defect)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox155 | --- | fixed |
People
(Reporter: leggert, Assigned: juliana)
References
(Blocks 2 open bugs)
Details
(Keywords: blocked-ux, Whiteboard: [fidefe-post-nova])
Attachments
(3 files)
| Reporter | ||
Comment 1•3 months ago
|
||
| Reporter | ||
Comment 3•3 months ago
•
|
||
The first is just the normal macOS window border and the Nova insert.
The second is the bookmark bar.
Triage (Nova / browser.nova.enabled): this is a rounded-corner mismatch in the Nova chrome. I couldn't render the two attached screenshots in my environment, so I localized from the CSS; please confirm which elements the shots show.
Nova's rounded "blocks" (.chrome-block: #navigator-toolbox, #sidebar-container/#sidebar-box, and the content area) get border-radius: var(--chrome-block-radius) + a 1px solid border with overflow: clip (browser/themes/shared/browser-shared.css:301-308). --chrome-block-radius is 10px (default) / 4px (Windows) / 8px (Mica) / 16px (macOS Tahoe). Corner mismatches here come from adjacent/nested radii that don't agree.
Most concrete inconsistency I found: in browser/themes/shared/tabbrowser/tabs.css, tab-split-view-wrapper is border-radius: var(--tab-border-radius) (= --button-border-radius = --border-radius-medium = 8px) at ~line 1458, but is overridden to var(--border-radius-large) (16px) at ~lines 1470 and 1491, while the child .tabbrowser-tabs stay at 8px — so the wrapper's corners don't follow the tab corners.
Proposed approach: make nested radii concentric/consistent — pick one radius token for the split-view wrapper and its child tabs (drop the stray 16px overrides, or reduce the tab radius to match), and for bordered .chrome-blocks ensure inner clipped children use --chrome-block-radius minus the 1px border so the border and inner corner align. Target files: browser/themes/shared/tabbrowser/tabs.css, browser/themes/shared/tabbrowser/content-area.css, browser/themes/shared/browser-shared.css.
No existing test covers these radii (split-view tests under browser/components/tabbrowser/test/browser/ are functional only). Please confirm from the screenshots which elements are mismatched so the fix can be scoped precisely.
This is an automated analysis result. If this result is incorrect please add a needinfo and feel free to correct the error.
Hi Lars, thanks for the info. Could you please provide more steps to reproduce the issue?
| Reporter | ||
Comment 6•2 months ago
•
|
||
There is nothing to reproduce. You look at any web page in a window.l for the first. For the second, you turn on the bookmark bar.
Updated•2 months ago
|
Comment 7•2 months ago
|
||
Screenshot 1: The outer border is drawn by macOS, the inner border is from a chrome block (content area or sidebar). This is a clear example of how concentric rounded corners with the same radius do not look quite right -- inner rounded corners need to have lower border radius to look good.
Screenshot 2: The outer border is from a chrome block (bookmarks toolbar when that toolbar is the bottom visible toolbar) and the "inner border" is the border around an individual bookmark. This is not quite concentric rounded corners because the first bookmark is also offset from the edge by a margin.
Either way, will raise to UX.
Updated•2 months ago
|
Updated•2 months ago
|
| Assignee | ||
Comment 8•2 months ago
•
|
||
these are two separate issues, I filed 2056624 for the latter one.
Updated•2 months ago
|
| Assignee | ||
Comment 9•2 months ago
|
||
Updated•2 months ago
|
Comment 10•2 months ago
|
||
Comment 11•2 months ago
|
||
| bugherder | ||
Updated•1 month ago
|
Description
•