Skip to content

Commit

Permalink
fix: icon alignment for all [data-icon] (#2271)
Browse files Browse the repository at this point in the history
  • Loading branch information
nmerget authored Feb 29, 2024
1 parent 86e073d commit d4b3fa4
Show file tree
Hide file tree
Showing 7 changed files with 8 additions and 2 deletions.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
2 changes: 0 additions & 2 deletions packages/components/src/components/icon/icon.scss
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,6 @@
.db-icon {
@include icons.is-icon-text-replace();

display: flex;

/* Safari hack */
@supports (-webkit-hyphens: none) {
display: inline-block;
Expand Down
8 changes: 8 additions & 0 deletions packages/foundations/scss/icons/include.scss
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,10 @@

[data-icon],
[data-icon-before] {
&:is(span) {
display: inline-flex;
}

&::before {
@extend %icon !optional;
margin-inline-end: icon-helpers.$default-icon-margin-end;
Expand All @@ -65,6 +69,10 @@
}

[data-icon-after] {
&:is(span) {
display: inline-flex;
}

&::after {
@extend %icon !optional;
content: icon-helpers.$default-icon-content-after;
Expand Down

0 comments on commit d4b3fa4

Please sign in to comment.