-
Notifications
You must be signed in to change notification settings - Fork 970
Conversation
@bsclifton thanks for the check... It looks like the lock icon is moving up, and appears to be larger. Do you have a separate windows adjustment for it? |
@bradleyrichter I don't see any Windows specific styles being applied. Lock shows as 16px (it's the FontAwesome face w/ class "fa-lock") Here's a closer up shot. It does seem the URL could be centered better |
Here's where the margin comes from (on Windows) that pushes the text in the input down: browser-laptop/less/navigationBar.less Line 731 in f62306a
Setting it to have 0 margin on top seems to fix the issue (although, I suspect it may break Mac): |
@bradleyrichter if you want to resolve the conflict, I think this is good for merge 😄 The issue I pointed out above is already captured with #4814 |
3df32ac
to
b78a280
Compare
Test Plan:
Confirm styling and alignments changes are OK on windows 7, 8, 10.