brave / brave-browser

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

Move advanced `vertical tab` settings into `brave://settings/appearance` #29843

Closed kjozwiak closed 1 year ago

kjozwiak commented 1 year ago

Description

As discussed via Slack, we should move the more advanced settings for vertical tabs into brave://settings/appearance rather than displaying them within the right click context menu. For example, moving Show title bar & Float on mouseover into brave://settings/appearance.

Steps to Reproduce

  1. install 1.52.54 Chromium: 112.0.5615.121
  2. right click on a tab and select Use vertical tabs
  3. once selected, notice the advanced settings such as Show title bar & Float on mouseover
  4. these and anything else related to vertical tabs should be moved into brave://settings/appearance

Actual result:

image

Expected result:

We should move the more advanced settings into brave://settings/appearance rather than displaying them via the context menu.

https://www.figma.com/file/tLXWGCpNoiJxDZDdpfordj/Desktop-Settings?node-id=3383-67555&t=nZ9X4L2m1MUuTamn-0

Reproduces how often:

100% reproducible using the STR/Cases outlined above.

Brave version (brave://version info)

Brave | 1.52.54 Chromium: 112.0.5615.121 (Official Build) nightly (64-bit)
-- | --
Revision | 39cc4e45904ae9f1741c4fbaa866e629c96f2268-refs/branch-heads/5615_51@{#8}
OS | Windows 11 Version 22H2 (Build 22621.1555)

Version/Channel Information:

Other Additional Information:

Miscellaneous Information:

CCing @rebron

kjozwiak commented 1 year ago

The above requires 1.51.106 or higher for 1.51.x verification 👍

MadhaviSeelam commented 1 year ago

Verification PASSED using

Brave | 1.51.106 Chromium: 113.0.5672.53 (Official Build) (64-bit)
-- | --
Revision | 12f5dac35d12e8f4e72d7dd11df557ef93bc046f-refs/branch-heads/5672@{#703}
OS | Windows 11 Version 22H2 (Build 22621.1555)

Case 1: Enable #brave-vertical-tabs in brave://flags

  1. Enabled #brave-vertical-tabs and clicked Relaunch
  2. opened brave://settings/appearance

    • Confirmed following is shown under Tabs section:
      • Confirmed Use vertical tabs settings is displayed and the toggle is disabled as a default
      • Confirmed Show title bar setting is displayed under Use vertical tabs section and unchecked as a default
      • Confirmed Float on mouseover setting is displayed under Use vertical tabs section and is checked as a default
      • Additionally, `Disable mute function in tab speaker icons & Tab hover mode settings are shown
    • Confirmed when Use vertical tabs is enabled in brave://settings/appearance, vertical tab view is shown
    • Confirmed Show title bar & Float on mouseover settings are no longer shown in the context menu
    • Confirmed Use vertical tabs is selected in the context menu
    • Confirmed Float on mouseover is working as expected in the Vertical tab view when checked
    • Confirmed when Float on mouseover unchecked, vertical tab view is shown in expanded/minimized view
    • Confirmed when Show title bar is checked, title is shown
Example Example Example Example Example
image image image image image

Case 2: Disable #brave-vertical-tabs in brave://flags

  1. Disabled #brave-vertical-tabs and clicked Relaunch
  2. opened brave://settings/appearance
Example Example Example
image image image
LaurenWags commented 1 year ago

Added QA/Test-All-Platforms label per discussion with @MadhaviSeelam as the test plan from @sangwoo108 calls out all OSes.

stephendonner commented 1 year ago

Verification PASSED using

Brave 1.51.106 Chromium: 113.0.5672.53 (Official Build) (x86_64)
Revision 12f5dac35d12e8f4e72d7dd11df557ef93bc046f-refs/branch-heads/5672@{#703}
OS macOS Version 11.7.6 (Build 20G1231)

Shared Steps:

- installed `1.51.106` - launched Brave - opened `brave://flags` in a new tab

Case 1: Enable #brave-vertical-tabs in brave://flags - PASSED

1. Enabled `#brave-vertical-tabs` and clicked Relaunch 2. opened `brave://settings/appearance` - Confirmed following is shown under `Tabs` section: - Confirmed `Use vertical tabs` settings is displayed and the toggle is disabled as a default - Confirmed `Show title bar` setting is displayed under `Use vertical tabs` section and unchecked as a default - Confirmed `Float on mouseover` setting is displayed under `Use vertical tabs` section and is checked as a default - Additionally, `Disable mute function in tab speaker icons & Tab hover mode settings are shown - Confirmed when `Use vertical tabs` is enabled in `brave://settings/appearance`, vertical tab view is shown - Confirmed `Show title bar` & `Float on mouseover` settings are no longer shown in the context menu - Confirmed `Use vertical tabs` is selected in the context menu - Confirmed `Float on mouseover` is working as expected in the Vertical tab view when checked - Confirmed when `Float on mouseover` unchecked, vertical tab view is shown in expanded/minimized view - Confirmed when `Show title bar` is checked, title is shown Example | Example | Example | Example | Example ---- | ----- | ----- | ------ | ------- Screen Shot 2023-04-27 at 2 39 47 PM | Screen Shot 2023-04-27 at 2 40 21 PM | Screen Shot 2023-04-27 at 2 43 08 PM | Screen Shot 2023-04-27 at 2 43 23 PM | Screen Shot 2023-04-27 at 2 43 34 PM

Case 2: Disable #brave-vertical-tabs in brave://flags - PASSED

1. Disabled `#brave-vertical-tabs` and clicked Relaunch 2. opened `brave://settings/appearance` - Confirmed `Use vertical tabs` setting is no longer shown in the `Tabs` section - Confirmed `Show title bar` & Float on mouseover` under `Use vertical tabs` section is not shown - Confirmed Vertical tab view is no longer shown from case 1 and switched to horizonal tab view - Confirmed `Use vertical tabs` settings is not available in the context menu Example | Example | Example ---- | ----- | ----- Screen Shot 2023-04-27 at 2 49 18 PM | Screen Shot 2023-04-27 at 2 49 51 PM | Screen Shot 2023-04-27 at 2 50 03 PM

Verification passed on

Brave 1.51.107 Chromium: 113.0.5672.63 (Official Build) (64-bit)
Revision 0e1a4471d5ae5bf128b1bd8f4d627c8cbd55f70c-refs/branch-heads/5672@{#912}
OS Ubuntu 18.04 LTS

Shared Steps:

- installed `1.51.107` - launched Brave - opened `brave://flags` in a new tab

Case 1: Enable #brave-vertical-tabs in brave://flags - PASSED

1. Enabled `#brave-vertical-tabs` and clicked Relaunch 2. opened `brave://settings/appearance` - Confirmed following is shown under `Tabs` section: - Confirmed `Use vertical tabs` settings is displayed and the toggle is disabled as a default - Confirmed `Show title bar` setting is displayed under `Use vertical tabs` section and unchecked as a default - Confirmed `Float on mouseover` setting is displayed under `Use vertical tabs` section and is checked as a default - Additionally, `Disable mute function in tab speaker icons & Tab hover mode settings are shown - Confirmed when `Use vertical tabs` is enabled in `brave://settings/appearance`, vertical tab view is shown - Confirmed `Show title bar` & `Float on mouseover` settings are no longer shown in the context menu - Confirmed `Use vertical tabs` is selected in the context menu - Confirmed `Float on mouseover` is working as expected in the Vertical tab view when checked - Confirmed when `Float on mouseover` unchecked, vertical tab view is shown in expanded/minimized view - Confirmed when `Show title bar` is checked, title is shown Example | Example ---- | ----- ![image](https://user-images.githubusercontent.com/34715963/235424116-279f532f-329b-4e20-af66-824eb71cb2fc.png)|![image](https://user-images.githubusercontent.com/34715963/235424137-0aa65c9c-c29c-413c-a431-c3d24186a5e5.png)

Case 2: Disable #brave-vertical-tabs in brave://flags - PASSED

1. Disabled `#brave-vertical-tabs` and clicked Relaunch 2. opened `brave://settings/appearance` - Confirmed `Use vertical tabs` setting is no longer shown in the `Tabs` section - Confirmed `Show title bar` & Float on mouseover` under `Use vertical tabs` section is not shown - Confirmed Vertical tab view is no longer shown from case 1 and switched to horizonal tab view - Confirmed `Use vertical tabs` settings is not available in the context menu Example | Example ---- | ----- ![image](https://user-images.githubusercontent.com/34715963/235424249-79bce5fa-8987-4e6a-96ef-99387225c81d.png)|![image](https://user-images.githubusercontent.com/34715963/235424261-e093d84b-462d-4bd0-9897-35ef2ac5a9bf.png)