-
Notifications
You must be signed in to change notification settings - Fork 972
Commit
- this adds some needed space above and below the text string - increases tab text to 12 on mac (already 12 on windows) - tabs buttons adjusted to align properly auditors: @bbondy @bsclifton
- Loading branch information
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -22,8 +22,8 @@ span.buttonSeparator { | |
background-color: @buttonColor; | ||
display: inline-block; | ||
width: 20px; | ||
height: 12px; | ||
margin: 8px 4px 0 0; | ||
height: 13px; | ||
margin: 9px 4px 0 0; | ||
cursor: default; | ||
-webkit-mask-image: url('../img/toolbar/menu_btn.svg'); | ||
-webkit-mask-repeat: no-repeat; | ||
|
@@ -68,7 +68,7 @@ span.buttonSeparator { | |
line-height: 18px; | ||
width: 18px; | ||
height: 18px; | ||
font-size: 12px; | ||
font-size: 24px; | ||
This comment has been minimized.
Sorry, something went wrong.
This comment has been minimized.
Sorry, something went wrong.
bradleyrichter
Author
Contributor
|
||
} | ||
|
||
&.primaryButton, | ||
|
4 comments
on commit 615a6ec
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
++
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
funny how 2px can make such a difference.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks extra good on Windows 😄
@bradleyrichter would you please let me know what this change was needed for? This seems to be breaking .smallbutton on
about:styles