Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Refine Toolbar UI with New Icons and styling refinements #13981

Closed
bradleyrichter opened this issue May 1, 2018 · 10 comments
Closed

Refine Toolbar UI with New Icons and styling refinements #13981

bradleyrichter opened this issue May 1, 2018 · 10 comments

Comments

@bradleyrichter
Copy link
Contributor

Issue: Dreams of UI refinements stuck in design world.

Fix: Implement collection of UI design refinements and new icon set.

Test proceedure:

Open new instance that includes new changes.
Compare to previous version.
Be amazed by beauty of new version.
Click buttons and resize window to make sure nothing blows up or looks wonky.

actual screen shot showing changes from @petemill go here:

@bradleyrichter bradleyrichter added design A design change, especially one which needs input from the design team. initiative/ux-harmonization labels May 1, 2018
@bradleyrichter bradleyrichter added this to the 0.24.x (Nightly Channel) milestone May 1, 2018
@alexwykoff alexwykoff modified the milestones: 0.24.x (Nightly Channel), 0.23.x (Developer Channel) May 8, 2018
@petemill
Copy link
Member

petemill commented May 22, 2018

Initial aims

  • top nav height / alignment polish
  • Url bar style: titlemode, non-titlemode, :hover, :focus, :active
  • Url bar loading progress bar
  • Navigation icons update
  • Preferences icons update
  • Tab bar style update
  • Tab loading icon
  • Preview build 1
    • Style updates from build 1 feedback
  • Preview build 2
    • Style updates from build 2 feedback
  • Preview build 3
    • Style updates from build 3 feedback

Additional aims

  • Icons v3 with explicit box sizing inside SVG
  • Ditch faded unloaded tab
  • Icon hover / active states on extension buttons, nav buttons and main menu button
  • fix weird url bar width behavior
  • fix solid tab border color cutting off favicon glow

Dependencies

  • v1 design sketch
  • icons
  • Shields icon spec
  • BAT icon spec
  • design spec or 👍on hover/active nav button styles
  • design spec or 👍on hover/active new tab button styles
  • design spec or 👍on hover/active extension button styles
  • Security 👍 on lock icon / EV text color

Test plan

URL bar progress bar

  • URL bar shows loading progress bar during tab load
  • URL bar does not show loading progress bar when tab is not loading

Icons

  • Preferences navigation icons are new
  • Top navigation bar icons are new
  • All top navigation bar icons have hover and mousedown states (as depicted in below comment)
  • Top navigation bar icons and url bar still fit when screen narrows to minimum width
  • Tab loading animation is new spinner
  • Tab loading animation fits on very small tabs at minimum width
  • URL bar lock icon is grey local for https
  • URL bar lock icon is green local for https with EV cert
  • URL bar lock icon is red unlock for http / other
  • URL bar icon is custom search engine for ':a hi' style search
  • Menu icon is on top navigation bar (no longer next to tabs)
  • When many extensions are enabled, the top nav icons, and url bar all fit gracefully

Payments button

  • Payments button shows as disabled on about:* pages
  • Payments button shows as enabled but inactive when first visiting a site
  • Payments button automatically turns to enabled and active after 8 seconds
  • Payments button can be clicked active / inactive when enabled

Shields button

  • Notification count is centered within background 'bubble'
  • Notification count gracefully fits a high number of blocked entries

@petemill
Copy link
Member

petemill commented May 22, 2018

Status as of beta:

image

Title mode

image

Status as of preview2:

Url bar

Title mode

image

Url mode

image

Hover

image

Focus

image

Icon variations

image

Tabs bar (click to enlarge)

image

image

New Tab Button

image
Hover
image
Mouse down
image

Nav buttons

image

Hover

image

Mouse down

image

All the buttons

image

image

image
image
image

Status as of preview1

image

image

@kjozwiak
Copy link
Member

@petemill @bradleyrichter other than the changes outlined in #13981 (comment), is there anything else QA needs to do? Do we have a design document that outlines the behaviour of the new UI changes anywhere?

@petemill
Copy link
Member

@kjozwiak no spec, it has been an iterative process. I believe we have approval from the design team along for the UX, with the included and pending iterative requests which are logged as separate issues, as of 0.23.14

@petemill
Copy link
Member

Added screenshots at #13981 (comment) to verify notable behavior.

@petemill
Copy link
Member

Added test plan to #13981 (comment)

@srirambv
Copy link
Collaborator

@petemill Is this expected with the new icon set?
shields click

@petemill
Copy link
Member

petemill commented Jun 25, 2018 via email

@bradleyrichter
Copy link
Contributor Author

bradleyrichter commented Jun 25, 2018 via email

@srirambv
Copy link
Collaborator

srirambv commented Jun 27, 2018

Verified on Windows 10 x64 using

  • 0.23.19 - 178c3fb
  • Muon - 7.1.3
  • libchromiumcontent - 67.0.3396.87

Verified with macOS 10.12.6 using

  • 0.23.19 178c3fb
  • Muon 7.1.3
  • libchromiumcontent 67.0.3396.87

Verified on Ubuntu 17.10 x64

  • 0.23.19 178c3fb
  • Muon 7.1.3
  • libchromiumcontent: 67.0.3396.87

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

No branches or pull requests

9 participants