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

[Desktop] design updates to customize dialog follow up to #9455 #10318

Closed
10 tasks
rebron opened this issue Jun 16, 2020 · 4 comments · Fixed by brave/brave-core#6015
Closed
10 tasks

[Desktop] design updates to customize dialog follow up to #9455 #10318

rebron opened this issue Jun 16, 2020 · 4 comments · Fixed by brave/brave-core#6015

Comments

@rebron
Copy link
Collaborator

rebron commented Jun 16, 2020

Description

Design follow-up to #9455.

  • Remove Show from the section title - moved to [Desktop] Remove "Show" from NTP Settings tab labels #10591
  • Remove blue focus border when selection is selected
  • Remove alt text Customize dashboard
  • Add scrim in the background (see design)
  • Widget title, description and button text should all be Poppins
  • Use + icon for Add; button
  • Remove bottom margin for all sections, not just More cards
  • Add missing widget stack icon
  • Only show message when 5/5 of cards are used.
  • Use right opacity for card titles (see design)

Design/Expected Result

design_qa_2
design_qa_3
design_qa_4
design_qa_5

Reproduces how often:

n/a

Brave version (brave://version info)

Address fixes for Brave 1.11.x

Version/Channel Information:

  • Can you reproduce this issue with the current release? No
  • Can you reproduce this issue with the beta channel? Yes
  • Can you reproduce this issue with the dev 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:

Thanks @karenkliu

@rebron
Copy link
Collaborator Author

rebron commented Aug 5, 2020

@ryanml 1.13.x milestone right?
@karenkliu When you get a chance, can you see if this all looks good to you? Can test in beta with 1.13.14 or higher or any Nightly build?

@rebron rebron closed this as completed Aug 5, 2020
@rebron rebron added this to the 1.13.x - Beta milestone Aug 5, 2020
@karenkliu
Copy link

@rebron @ryanml We're still missing a couple design fixes from the first round:

design QA 6
design QA 7

@ryanml
Copy link
Contributor

ryanml commented Aug 5, 2020

@rebron yes 1.13 - I'll make the fixes @karenkliu mentioned here then uplift them

@LaurenWags
Copy link
Member

LaurenWags commented Aug 24, 2020

Verified passed with

Brave	1.13.79 Chromium: 85.0.4183.69 (Official Build) dev (64-bit)
Revision	4554ea1a1171bd8d06951a4b7d9336afe6c59967-refs/branch-heads/4183@{#1426}
OS	macOS Version 10.14.6 (Build 18G3020)

Per https://bravesoftware.slack.com/archives/CQUHQ0PM3/p1598038077005900?thread_ts=1598037921.005400&cid=CQUHQ0PM3, design changes have already been reviewed by @karenkliu

Spot checked some design elements that were listed in brave/brave-core#6351 (comment) such as

  • confirmed that once the Customize modal is visible, that there's an overlay over the NTP that looks a bit darker to emphasis the Customize modal.
  • confirmed that clicking anywhere on the NTP page dismisses the Customize modal without issues
  • confirmed focus order uses Brave color (not blue)

Generally confirmed functionality of customize dashboard works

  • able to toggle items off/on via Customize Dashboard
  • confirmed these selections were reflected on brave://settings/newTab
  • confirmed able to toggle items from brave://settings/newTab and have the status reflected on Customize Dashboard
  • confirmed selections were retained on browser restart

Encountered #11374


Verification passed on


Brave | 1.13.79 Chromium: 85.0.4183.69 (Official Build) dev (64-bit)
-- | --
Revision | 4554ea1a1171bd8d06951a4b7d9336afe6c59967-refs/branch-heads/4183@{#1426}
OS | Windows 10 OS Version 1903 (Build 18362.1016)


Verification passed on

Brave 1.13.82 Chromium: 85.0.4183.83 (Official Build) (64-bit)
Revision 94abc2237ae0c9a4cb5f035431c8adfb94324633-refs/branch-heads/4183@{#1658}
OS Ubuntu 18.04 LTS

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