Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update brave://settings/appearance with section titles and better grouping #33128

Closed
rebron opened this issue Sep 20, 2023 · 5 comments · Fixed by brave/brave-core#20409
Closed

Comments

@rebron
Copy link
Collaborator

rebron commented Sep 20, 2023

Description

Update brave://settings/appearance page with sub-titles and grouping.
This will bring the Brave Wallet button show/hide to the appearance page from web3 section.
Add section titles for Address bar, Tabs, Sidebar, and Content.
Figma link: https://www.figma.com/file/IXNCYwXmMWaF6u7Pdv1pOW/Desktop-settings---Leo?type=design&node-id=1-17&mode=design&t=k9wGm2Ix9QSrfSZB-0

Design:

Screenshot 2023-09-19 at 5 07 38 PM

Reproduces how often:

Easily.

Brave version (brave://version info)

Target 1.60.x

Version/Channel Information:

  • Can you reproduce this issue with the current release? yes
  • Can you reproduce this issue with the beta channel? yes
  • Can you reproduce this issue with the nightly channel? yes

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? n/a
  • Does the issue resolve itself when disabling Brave Rewards? n/a
  • Is the issue reproducible on the latest version of Chrome? n/a

Miscellaneous Information:

Related pr: brave/brave-core#20160

@rebron rebron changed the title Update brave://settings/appearance with sub-titles and grouping Update brave://settings/appearance with section titles and better grouping Sep 20, 2023
@rebron rebron added the priority/P3 The next thing for us to work on. It'll ride the trains. label Sep 20, 2023
@simonhong
Copy link
Member

Working on this.

@kjozwiak
Copy link
Member

@brave/qa-team you can use brave/brave-core#20409 (comment) as a template. The above requires 1.60.95 or higher for 1.60.x verification 👍

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Oct 18, 2023

Verification PASSED on

Brave | 1.60.95 Chromium: 118.0.5993.88 (Official Build) beta (64-bit)
-- | --
Revision | d65c9c374e175c06c50025db46cf953fd200e9cc
OS | Windows 10 Version 22H2 (Build 19045.3570)

Basically using the information & screenshots examples from brave/brave-core#20409 (comment), ensured the following:

Vertical Tabs using Light theme

Example Example
image image

Vertical Tabs using Dark theme

Example Example
image image

Horizontal Tabs using Light theme

Example Example
image image

Horizontal Tabs using Dark theme

Example Example
image image

Leo position (below Web3)

Example Example
image image

Removing Setting text from navigation via brave://settings

Old New
image image

Always show toolbar in fullscreen only appearing under macOS via Toolbar

@stephendonner
Copy link

stephendonner commented Oct 30, 2023

Verification PASSED using

Brave | 1.60.106 Chromium: 119.0.6045.59 (Official Build) (x86_64)
-- | --
Revision | 16064193dde8c55f64dabd32f125c0cd9380f96a
OS | macOS Version 14.2 (Build 23C5030f)

Basically using the information & screenshots examples from brave/brave-core#20409 (comment), ensured the following:

Vertical Tabs using Light theme

Example Example
Screenshot 2023-10-30 at 11 54 38 AM Screenshot 2023-10-30 at 11 53 55 AM

Vertical Tabs using Dark theme

Example Example
Screenshot 2023-10-30 at 11 55 46 AM Screenshot 2023-10-30 at 11 55 52 AM

Horizontal Tabs using Light theme

Example Example
Screenshot 2023-10-30 at 11 59 42 AM Screenshot 2023-10-30 at 11 59 53 AM

Horizontal Tabs using Dark theme

Example Example
Screenshot 2023-10-30 at 11 58 40 AM Screenshot 2023-10-30 at 11 58 36 AM

Leo position (below Web3)

Example Example
Screenshot 2023-10-30 at 12 01 38 PM Screenshot 2023-10-30 at 12 01 46 PM

Removing Setting text from navigation via brave://settings

Old New
Screenshot 2023-10-30 at 12 13 36 PM Screenshot 2023-10-30 at 12 06 26 PM

Always show toolbar in fullscreen only appearing under macOS via Toolbar

example example
Screenshot 2023-10-30 at 12 19 29 PM Screenshot 2023-10-30 at 12 19 45 PM

@btlechowski
Copy link

btlechowski commented Nov 1, 2023

Verified with

Brave 1.60.110 Chromium: 119.0.6045.105 (Official Build) (64-bit)
Revision 0bb6e7a81aed340b5a9e8b752c1cf92768257c40
OS Linux

Basically using the information & screenshots examples from brave/brave-core#20409 (comment), ensured the following:

Vertical Tabs using Light theme

Example Example
image image

Vertical Tabs using Dark theme

image image

Horizontal Tabs using Light theme

Example Example
image image

Horizontal Tabs using Dark theme

Example Example
image image

Leo position (below Web3)

Example Example
image

Removing Setting text from navigation via brave://settings

image image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants