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

Add separator for urlBarIconContainer where EVC is not available #13159

Closed
srirambv opened this issue Feb 16, 2018 · 2 comments
Closed

Add separator for urlBarIconContainer where EVC is not available #13159

srirambv opened this issue Feb 16, 2018 · 2 comments
Labels
feature/URLbar polish Nice to have — usually related to front-end/visual tasks. wontfix

Comments

@srirambv
Copy link
Collaborator

Description

Follow up of #13147 (comment)

Actual result:

Expected result:

Reproduces how often:

Brave Version

about:brave info:

Reproducible on current live release:

Additional Information

@srirambv srirambv added polish Nice to have — usually related to front-end/visual tasks. feature/URLbar labels Feb 16, 2018
@srirambv srirambv added this to the 0.23.x (Nightly Channel) milestone Feb 16, 2018
@cndouglas
Copy link

Maybe not needed if the following comment is still relevant.

#7888 (comment) @bradleyrichter:

a border? I like Chrome's "border". Invisible. ; )

@bsclifton bsclifton self-assigned this Feb 16, 2018
@bsclifton
Copy link
Member

Good catch, @liunkae! Let's close this issue for now

@MargarytaChepiga no need to add the border (per comment by our head of design, @bradleyrichter in the original issue)

@bsclifton bsclifton removed their assignment Feb 16, 2018
@bsclifton bsclifton removed this from the 0.23.x (Nightly Channel) milestone Feb 16, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature/URLbar polish Nice to have — usually related to front-end/visual tasks. wontfix
Projects
None yet
Development

No branches or pull requests

3 participants