Open aguscruiz opened 1 month ago
Border of active tabs is 1.5px
@aguscruiz real number may not work on Windows. Can we have integer for the thickness?
Border color of inactive tab has also been updated
And curious if Browser/ToolBar/Button/Outline is valid token. I couldn't find the color.
@aguscruiz real number may not work on Windows. Can we have integer for the thickness?
Yeah we can use 2px instead. We can use that on macOS/Linux aswell, so we keep the same in all. Btw the color token I used is for border/icon/interactive. That should solve the issue with the color not having contrast in dark mode
And curious if Browser/ToolBar/Button/Outline is valid token. I couldn't find the color.
This one is part of the new color tokens for the material design work. It's built on top of the new primitives, so it's not accessible without those.
You need it for the url indicator?
If that's the use case, you can use the same color we give the top tab bar background. It's the same value.
Note: With that URL indicator, the whole inactive tab gets a border color too
I left a clean design without images here so you can see it better https://www.figma.com/design/Dt0iJdVaaAj7DikWLO1P7I/Split-view-%2F-Sidebar-browsing?node-id=2318-16574&t=6FHr3SRo0nZmN0Fo-1
If that's the use case, you can use the same color we give the top tab bar background. It's the same value.
Yeah, I needed it for inactive web view's border. I guess that's the same value with the mini url bar's border.
Browser's current frame color looks lighter than the design guide, so vaguely notice the border 🤔
Yeah, another update from the material design Token update that's pending.
Okay then let me push as is 👍
I recorded this yesterday but still applies to 1.69.66 Chromium: 127.0.6533.26 (Official Build) nightly (64-bit)
Seems like this behavior appeared after Polish Split view UI brave/brave-core#24385
PR, and it is basically, that when you click on the tabs of the split view, the active tab will have 2px less, than when it is not the active one.
You can see it in devtools, how the size changes. Plus seems like devtools now disappears for a while when switching from non-devtools split view tab to the devtools one
And as can be seen, it happens in both VT and HT.
https://github.com/brave/brave-browser/assets/156715894/14e0ea9b-af65-4199-9a5a-90768aa2f7c1
https://github.com/brave/brave-browser/assets/156715894/3eb65f02-bdf1-4c46-ad3d-fa55b369bfdf
Description
Latest version of Figma
https://www.figma.com/design/Dt0iJdVaaAj7DikWLO1P7I/Split-view-%2F-Sidebar-browsing?node-id=348-28191&t=0r7ExpW2INc0KU4d-1
On hold
Tab bar
Omnibar
Page area
Contextual menu on gutter