-
Notifications
You must be signed in to change notification settings - Fork 16
feat(buttons): Add new design styles and improvements #91
Conversation
kendo-dropdownbutton.k-state-focused, | ||
kendo-dropdownbutton:focus { | ||
outline: 0; | ||
.k-dropdownbutton { |
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.
If we have k-split-button
, then this should be k-dropdown-button
, or vice versa -- k-splitbutton
and k-dropdownbutton
.
The rest is fine.
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.
Seems to be the same question that we had for the button-group
and buttongroup
, no? telerik/kendo-angular-buttons#45
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.
Changed the class name to k-dropdown-button.
|
||
$primary-focused-box-shadow: 0 0 0 2px lighten($primary-button-bg, 23%) !default; | ||
$primary-active-box-shadow: inset 0 3px 5px desaturate($primary-button-bg, 3%) !default; | ||
$primary-focused-box-shadow: 0 3px 4px 0 rgba(255, 99, 88, .06) !default; |
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.
How is this color calculated? seems related to $accent
.
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.
It is exactly the $accent color. I was unaware that I could use a hex variable inside an RGBA function in SASS. Corrected now.
No description provided.