-
Notifications
You must be signed in to change notification settings - Fork 7.5k
Style Extension Update buttons to the match the green notification "badge" #6315
Changes from 1 commit
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -1070,6 +1070,22 @@ input[type="color"], | |
| outline: none; | ||
| } | ||
| } | ||
|
|
||
| // Update Button Type | ||
| &.update { | ||
| background-color: #91cc41; | ||
| border-color: #658e2d; | ||
| box-shadow: inset 0 1px 0 #d9edbd; | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Use rgba(255, 255, 255, 0.27) for box-shadow. |
||
| color: #fff; | ||
| text-shadow: 0 1px 0 #333; | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 0 -1px 0 rgba(0, 0, 0, 0.24) for text-shadow, the text is white so we need to invert the shadow. Button labels in Brackets are sunken.
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Add font-weight: 500; to increase the contrast a bit more between text and background color.
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Please use |
||
|
|
||
| &:active { | ||
| background-color: #82b839; | ||
| border-color: #5b8028; | ||
| box-shadow: inset 0 1px 0 #d3e6ba; | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Use rgba(0, 0, 0, 0.06) for pressed button shadow. |
||
| color: #fff; | ||
| } | ||
| } | ||
|
|
||
| // Button Sizes | ||
| &.large { | ||
|
|
||
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.
Let's use #65A015 for border-color.