Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

[Bug] Inconsistent icon spacing & icon to text spacing in URL bar #5923

Closed
Cheap-Skate opened this issue Oct 10, 2019 · 4 comments
Closed

[Bug] Inconsistent icon spacing & icon to text spacing in URL bar #5923

Cheap-Skate opened this issue Oct 10, 2019 · 4 comments
Labels
🐞 bug Crashes, Something isn't working, ..

Comments

@Cheap-Skate
Copy link

Cheap-Skate commented Oct 10, 2019

Latest version has Tracking Protection shield icon and padlock icon.

(1) spacing between shield and padlock icon is larger than between tabs and 3 dot menu icon. It looks funny.

(2) spacing from padlock icon to URL text is larger than between the search favicon and the search text on the Search screen. It should be the same in both cases.

Spacings should be consistent between icons on the URL bar, and between text spacing in the normal URL bar and the Search text bar.

IMHO the narrower spacing should be used in both cases, to win back some room for the URL text.

icon spacing 1

┆Issue is synchronized with this Jira Task

@Cheap-Skate Cheap-Skate added the 🐞 bug Crashes, Something isn't working, .. label Oct 10, 2019
@Dunexus
Copy link

Dunexus commented Oct 10, 2019

Related #3727

@NotWoods NotWoods added the needs:UX-feedback Needs UX Feedback label Nov 22, 2019
@brampitoyo brampitoyo self-assigned this Nov 25, 2019
@brampitoyo
Copy link

@Cheap-Skate Our design contains these measurements:

It does seem like the spacing between shield and padlock is indeed larger, as you noted.

Let’s make sure that we fix this according to the measurements above.

@brampitoyo brampitoyo removed their assignment Nov 25, 2019
@brampitoyo brampitoyo removed the needs:UX-feedback Needs UX Feedback label Nov 25, 2019
@Cheap-Skate
Copy link
Author

Part of the problem was that I was using Dark mode. Until recently, in Dark mode the vertical divider between shield and padlock was not visible on my Galaxy S7 so the spacing looked odd. Now the vertical divider is much more visible in Dark mode and the spacing does not look so strange.

@brampitoyo
Copy link

@Cheap-Skate Thanks for confirming! Since everything now seems to look okay, I will go ahead and close this issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🐞 bug Crashes, Something isn't working, ..
Projects
None yet
Development

No branches or pull requests

4 participants