brave / brave-browser

Brave browser for Android, iOS, Linux, macOS, Windows.
https://brave.com
Mozilla Public License 2.0
17.52k stars 2.27k forks source link

Update horizontal tab shape for pins, normal tabs, and groups #31646

Closed rebron closed 11 months ago

rebron commented 1 year ago

Description

Add padding all around individual pins, tabs, and grouped tabs. Adding the border all around will make the bottom border appear to "detach" tabs from the main content window and will allow for better highlighting/color grouping of grouped tabs.

Add border around pinned tabs.

Design

Screenshot 2023-07-14 at 8 56 48 AM

Figma: https://www.figma.com/file/H11ZOl6JMYbCTW4ZJXqR5V/%F0%9F%A6%81-Browser?type=design&node-id=125%3A9048&mode=design&t=nhfxV22NiGDSYN2e-1

Reproduces how often:

n/a

Brave version (brave://version info)

n/a

Version/Channel Information:

Other Additional Information:

Miscellaneous Information:

ghost commented 1 year ago

I like this new tab design, can't wait to see this happening in future releases.

There is also a request in community page to make brave use chrome like rounded corners tab UI on horizontal view. It would be really nice if there is a flag to enable chrome like rounded corners which use to exist earlier, many people have requested that for years now and it will feel more like home when chrome users come to brave when the manifest v3 is enforced.

I would really appreciate if you guys reconsider on adding that UI as an opt in basis and give power to change tab design in users hands so users can make it more personal like we do in Firefox.

rebron commented 1 year ago

@dpthegrey Is there a specific feature in Firefox with the Tabs UI that you're wanting to see?

ghost commented 1 year ago

Hello @rebron , With Firefox we can change the tabs shape and in general customise the browser's appearance using CSS. The FirefoxCSS Store which showcases what the community has built to customize the shape of Firefox tabs and themes. I would like to see Brave offer similar customisation options.

and/or if we could include a few static tab shapes that are widely used in common browsers, in addition to the proposed design above. These shapes could be available in the settings menu, in a dropdown format just like there use to be a flag which could change tabs shape in earlier versions.

Tab shape in Firefox

image

Tab shape in Google Chrome

image

MadhaviSeelam commented 11 months ago

Verification PASSED using

Brave | 1.60.81 Chromium: 118.0.5993.54 (Official Build) beta (64-bit)
-- | --
Revision | 1efb3f333eb41cc34af46ca31fb33c30e0afbfae
OS | Windows 11 Version 22H2 (Build 22621.2283)

and

Brave | 1.60.88 Chromium: 118.0.5993.70 (Official Build) beta (64-bit)
-- | --
Revision | ff9150ac5dd9934a7f431ddf478ad3e45ae68546
OS | Windows 11 Version 22H2 (Build 22621.2428)
  1. Installed 1.60.81
  2. launched Brave
  3. opened brave://flags
  4. enabled #brave-horizontal-tabs-update flag
  5. opened few tabs

Normal Window

- clicked on a tab to make the tab active in the normal window - hover over inactive tabs - compared when #brave-horizontal-tabs-update` flag disabled vs enabled #### Confirmed active tab shown with bottom border appeared to be `detached` from the main content window #### Confirmed inactive tab, when hovered over, shown with bottom border appeared to be `detached` from the main content window step 3 | step 6 | step 7 ----- | ---- | ----- image|image|image `#brave-horizontal-tabs-update` flag: Default (disabled)| Enabled ----- | ----- ![image](https://github.com/brave/brave-browser/assets/98358127/0d19fd9c-76f0-4fc2-b6e1-545c5fb5f829)|![image](https://github.com/brave/brave-browser/assets/98358127/7f4443eb-ec91-4823-aada-595e57e680a6) Dark theme: step 3 | step 6 | step 7 ----- | ---- | ----- ![image](https://github.com/brave/brave-browser/assets/98358127/c53bf9af-7782-42ea-80e5-a18bf23283a4)|![image](https://github.com/brave/brave-browser/assets/98358127/78e19ffc-4d2b-4542-bbde-43b42b0de5b1)|![image](https://github.com/brave/brave-browser/assets/98358127/11572a48-fd83-4808-8c52-c474e85cb193) `#brave-horizontal-tabs-update` flag: Default (disabled)| Enabled ----- | ----- ![image](https://github.com/brave/brave-browser/assets/98358127/3acb4b13-922a-46b1-a5f3-b78143c4d71b)|![image](https://github.com/brave/brave-browser/assets/98358127/cf74efac-ac6f-40d0-8595-7315236e779b)

Pinned tabs

- selected Pin tab from the context menu - pinned 3 tabs - compared when #brave-horizontal-tabs-update` flag disabled vs enabled #### Confirmed border around pinned tabs are shown and the shape is shown as per the Figma example | example ----- | ------ image|image `#brave-horizontal-tabs-update` flag: Disabled | Enabled ----- | ------ image|image Dark theme example | example ----- | ------ ![image](https://github.com/brave/brave-browser/assets/98358127/3870cb0c-5c4c-4954-bb5c-59c683c8a6c9)|![image](https://github.com/brave/brave-browser/assets/98358127/e58b968e-1e2b-4747-b73f-abc0baf9690d) `#brave-horizontal-tabs-update` flag: Disabled | Enabled ----- | ------ ![image](https://github.com/brave/brave-browser/assets/98358127/ecf17189-7a07-455d-9c97-2282869420c5)|![image](https://github.com/brave/brave-browser/assets/98358127/acd18cd9-2bad-4fe8-a5bc-f78394128633)

Grouped tabs

- selected `Add tab to new group` >> New group - Name the group (eg. newspapers) - Add each tab to the group (eg. newspapers) via context menu - hover over the group to view the added tabs - click on the group expand or collapse the group - compared when #brave-horizontal-tabs-update` flag disabled vs enabled #### Confirmed highlighting/color grouping of grouped tabs. shown as per the Figma example | example | example |example | example | example -----| ----- | ----- | ------ | ------ | ------- image|image|image|image|image|image `#brave-horizontal-tabs-update` flag: Disabled | Enabled ----- | ------ image|image Dark theme Example | Example | Example | Example ----- | ---- | ---- | ---- ![image](https://github.com/brave/brave-browser/assets/98358127/faac5bc4-51ef-4118-87ae-3a39ea9eae25)|![image](https://github.com/brave/brave-browser/assets/98358127/57cff904-ac84-4385-8b3e-cc6e3e1ae3ff)|![image](https://github.com/brave/brave-browser/assets/98358127/c940c139-958b-4035-9a98-f157b83674cd)|![image](https://github.com/brave/brave-browser/assets/98358127/562a14a9-45c6-40a5-8979-07f538865bb2)

New window

`#brave-horizontal-tabs-update` flag: Disabled | Enabled ----- | ------ image|image Dark theme Disabled | Enabled ----- | ------ image|image

Private window

`#brave-horizontal-tabs-update` flag: Disabled | Enabled ----- | ------ image|image Dark theme Disabled | Enabled ----- | ------ ![image](https://github.com/brave/brave-browser/assets/98358127/60bef212-41eb-4dd5-b4ca-e60a13d21752)|![image](https://github.com/brave/brave-browser/assets/98358127/5ecdbc85-a219-41f3-b144-ffca641c5a23)

Private window with Tor

`#brave-horizontal-tabs-update` flag: Disabled | Enabled ----- | ------ image|image Disabled | Enabled ----- | ------ ![image](https://github.com/brave/brave-browser/assets/98358127/82a6bc10-d61d-4107-80a8-34c56100b503)|![image](https://github.com/brave/brave-browser/assets/98358127/dfe06a21-d1f9-464f-8fdb-d4679439c0dc)

Upgrade

- Installed `1.58.137` - launched Brave - opened few tabs in the horizontal view with normal tabs, pinned tabs & grouped tabs - closed Brave - renamed the profile to `Brave-Browser-Beta` - Installed `1.60.81` - launched Brave - enabled `#brave-horizontal-tabs-update` in brave://flags #### Confirmed Pins tabs, normal tabs & grouped tabs are shown as expected and match to Figma example | example ----- | ----- image|image
LaurenWags commented 11 months ago

Verified with

Brave | 1.60.86 Chromium: 118.0.5993.54 (Official Build) beta (x86_64)
-- | --
Revision | a174cdee5ce1b329d409f3dd8c450e98b7ff4511
OS | macOS Version 13.6 (Build 22G120)

Shared Steps:

  1. Installed 1.60.x
  2. launched Brave
  3. closed and relaunched to pull griffin
  4. confirmed BraveHorizontalTabsUpdateEnabledStudy shows as enabled on brave://version
  5. opened few tabs

Normal Window

Continued from "Shared Steps": 6. clicked on a tab to make the tab active in the normal window 7. hover over inactive tabs #### Confirmed active tab shown with bottom border appeared to be `detached` from the main content window #### Confirmed inactive tab, when hovered over, shown with bottom border appeared to be `detached` from the main content window step 3 | step 6 | step 7 ----- | ---- | ----- 3 | 6 | 7 8. compared when study is enabled (100% on beta currently) vs. disabled (had to manually disable the study via flags) for both Dark and Light themes: Dark Theme: Active tab (study disabled) | Active tab (study enabled) | Inactive tab hover (study disabled) | Inactive tab hover (study enabled) --- | --- | --- | --- 1 | 2 | 3 | 4 Light Theme: Active tab (study disabled) | Active tab (study enabled) | Inactive tab hover (study disabled) | Inactive tab hover (study enabled) --- | --- | --- | --- 1 | 2 | 3 | 4

Pinned tabs

Continued from "Shared Steps": 6. selected Pin tab from the context menu 7. pinned 3 tabs #### Confirmed border around pinned tabs are shown and the shape is shown as per the Figma example | example ----- | ------ a | b 8. compared when study is enabled (100% on beta currently) vs. disabled (had to manually disable the study via flags) for both Dark and Light themes. (note, hovered over a pinned tab while the active tab was a non-pinned tab in the screenshots below) Study Disabled | Study Enabled --- | --- 1 | 2 3 | 4

Grouped tabs

Continued from "Shared steps 6. selected `Add tab to new group` >> New group 7. Name the group (eg. newspapers) 8. Add each tab to the group (eg. newspapers) via context menu 9. hover over the group to view the added tabs 10. click on the group expand or collapse the group #### Confirmed highlighting/color grouping of grouped tabs. shown as per the Figma example | example | example | example -----| ----- | ----- | ------ 1 | 2 | 3 | 4 11. compared when study is disabled vs enabled for both light and dark themes Study Disabled | Study Enabled --- | --- 1 | 1a 2 | 2a 3 | 3a 4 | 4a 5 | 5a 6 | 6a

New window

`#brave-horizontal-tabs-update` flag (enabled via griffin on Beta) for both Light and Dark themes: Disabled | Enabled ----- | ------ 1 | 1a 2 | 2a

Private window

`#brave-horizontal-tabs-update` flag (enabled via griffin on beta) Study Disabled | Study Enabled --- | --- 1 | 2

Private window with Tor

`#brave-horizontal-tabs-update` flag (enabled via griffin on beta) Study Disabled | Study Enabled --- | --- 1 | 2

Upgrade

- Installed `1.59.117` - launched Brave - opened few tabs in the horizontal view with normal tabs, pinned tabs & grouped tabs - closed Brave - renamed the profile to `Brave-Browser-Beta` - Installed `1.60.x` - launched Brave - confirmed `BraveHorizontalTabsUpdateEnabledStudy` shows as enabled on brave://version #### Confirmed Pins tabs, normal tabs & grouped tabs are shown as expected and match to Figma 1.59.x | 1.60.x ----- | ----- 1 59 | 1 60

Verified with

Brave 1.60.102 Chromium: 118.0.5993.96 (Official Build) beta (64-bit)
Revision 3598a9fc6b7752181feb25caa131bc386d6d054c
OS Linux

Shared Steps:

  1. Installed 1.60.x
  2. launched Brave
  3. closed and relaunched to pull griffin
  4. confirmed BraveHorizontalTabsUpdateEnabledStudy shows as enabled on brave://version
  5. opened few tabs

Normal Window

Continued from "Shared Steps": 6. clicked on a tab to make the tab active in the normal window 7. hover over inactive tabs #### Confirmed active tab shown with bottom border appeared to be `detached` from the main content window #### Confirmed inactive tab, when hovered over, shown with bottom border appeared to be `detached` from the main content window Light theme (https://github.com/brave/brave-browser/assets/34715963/81011558-619a-4cda-b7a3-91ba98f9ce45)|![image](https://github.com/brave/brave-browser/assets/34715963/457c5f8b-9e51-44f1-8d0c-8adc9c28d12e) --|-- Dark theme ![image](https://github.com/brave/brave-browser/assets/34715963/414cbff2-9a07-42d2-9093-f9cfbb52ee01)|![image](https://github.com/brave/brave-browser/assets/34715963/53085eb7-3277-4b28-af52-5ebde0119b4a) --|--

Pinned tabs

Continued from "Shared Steps": 6. selected Pin tab from the context menu 7. pinned 3 tabs #### Confirmed border around pinned tabs are shown and the shape is shown as per the Figma Light theme ![image](https://github.com/brave/brave-browser/assets/34715963/4b1ec3fd-9c5b-4c82-84c9-0935da9e8225) ![image](https://github.com/brave/brave-browser/assets/34715963/e0ef385e-965f-410e-93e0-856e16225ecb) Dark theme ![image](https://github.com/brave/brave-browser/assets/34715963/2c936fe2-5f5f-4f56-baec-3336193e9b34) ![image](https://github.com/brave/brave-browser/assets/34715963/9f53e58d-2753-4e70-81c0-6100d814696c)

Grouped tabs

Continued from "Shared steps 6. selected `Add tab to new group` >> New group 7. Name the group (eg. newspapers) 8. Add each tab to the group (eg. newspapers) via context menu 9. hover over the group to view the added tabs 10. click on the group expand or collapse the group #### Confirmed highlighting/color grouping of grouped tabs. shown as per the Figma ![image](https://github.com/brave/brave-browser/assets/34715963/bad49531-0d1a-41a3-a3e3-6dcf27022f0a) ![image](https://github.com/brave/brave-browser/assets/34715963/efd30546-17ea-41f4-b087-7537e5fa2eaf)

New window

![image](https://github.com/brave/brave-browser/assets/34715963/1b357471-0087-44ab-abda-ff82ce42f73f)

Private window

![image](https://github.com/brave/brave-browser/assets/34715963/e06965fc-f8c4-4bf3-9959-99b3a9d633c8)

Private window with Tor

![image](https://github.com/brave/brave-browser/assets/34715963/2e6f3eed-77a0-440c-af0e-40543247fa57)

Upgrade

- Installed `1.59.117` - launched Brave - opened few tabs in the horizontal view with normal tabs, pinned tabs & grouped tabs - closed Brave - renamed the profile to `Brave-Browser-Beta` - Installed `1.60.x` - launched Brave - confirmed `BraveHorizontalTabsUpdateEnabledStudy` shows as enabled on brave://version #### Confirmed Pins tabs, normal tabs & grouped tabs are shown as expected and match to Figma