Skip to content

Commit

Permalink
fix(button): update icon only ghost button colors (#5313)
Browse files Browse the repository at this point in the history
* fix(button): update icon only ghost button colors

* fix(ghost-button): update disabled state behavior

* fix(button): remove background when disabled
  • Loading branch information
tw15egan authored Feb 17, 2020
1 parent 61d1373 commit 8adc849
Show file tree
Hide file tree
Showing 2 changed files with 26 additions and 0 deletions.
25 changes: 25 additions & 0 deletions packages/components/src/components/button/_button.scss
Original file line number Diff line number Diff line change
Expand Up @@ -206,6 +206,31 @@
}
}

.#{$prefix}--btn--ghost.#{$prefix}--btn--icon-only
.#{$prefix}--btn__icon
path,
.#{$prefix}--btn--ghost.#{$prefix}--btn--icon-only .#{$prefix}--btn__icon {
fill: $icon-01;
}

.#{$prefix}--btn--ghost.#{$prefix}--btn--icon-only[disabled]
.#{$prefix}--btn__icon
path,
.#{$prefix}--btn--ghost.#{$prefix}--btn--icon-only[disabled]
.#{$prefix}--btn__icon {
fill: $disabled-02;
}

.#{$prefix}--btn--ghost.#{$prefix}--btn--icon-only[disabled] {
cursor: not-allowed;
}

.#{$prefix}--btn--ghost.#{$prefix}--btn--icon-only[disabled]
.#{$prefix}--assistive-text,
.#{$prefix}--btn--ghost.#{$prefix}--btn--icon-only[disabled]::before {
display: none;
}

.#{$prefix}--btn--icon-only.#{$prefix}--btn--tertiary {
padding-left: rem(15px);
padding-right: rem(15px);
Expand Down
1 change: 1 addition & 0 deletions packages/react/src/components/Button/Button-story.js
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,7 @@ const props = {
'Primary button (primary)': 'primary',
'Secondary button (secondary)': 'secondary',
'Tertiary button (tertiary)': 'tertiary',
'Ghost button (ghost)': 'ghost',
},
'primary'
),
Expand Down

0 comments on commit 8adc849

Please sign in to comment.