Skip to content

Commit 758dac4

Browse files
authored
Merge d462f0d into 3d7ce5a
2 parents 3d7ce5a + d462f0d commit 758dac4

File tree

6 files changed

+202
-189
lines changed

6 files changed

+202
-189
lines changed

.changeset/remove-base-icon.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
---
2+
"@patternfly/elements": major
3+
---
4+
`<pf-icon>`: Removed `BaseIcon` class. Reimplement (recommended) or extend `PfIcon`.

elements/package.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@
3030
"./pf-clipboard-copy/pf-clipboard-copy.js": "./pf-clipboard-copy/pf-clipboard-copy.js",
3131
"./pf-code-block/BaseCodeBlock.js": "./pf-code-block/BaseCodeBlock.js",
3232
"./pf-code-block/pf-code-block.js": "./pf-code-block/pf-code-block.js",
33-
"./pf-icon/BaseIcon.js": "./pf-icon/BaseIcon.js",
3433
"./pf-icon/pf-icon.js": "./pf-icon/pf-icon.js",
3534
"./pf-icon/icons/*": "./pf-icon/icons/*",
3635
"./pf-jump-links/pf-jump-links-item.js": "./pf-jump-links/pf-jump-links-item.js",

elements/pf-icon/BaseIcon.css

Lines changed: 0 additions & 22 deletions
This file was deleted.

elements/pf-icon/BaseIcon.ts

Lines changed: 0 additions & 163 deletions
This file was deleted.

elements/pf-icon/pf-icon.css

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,25 @@
1+
:host {
2+
position: relative;
3+
display: inline-block;
4+
line-height: 0;
5+
height: fit-content !important;
6+
width: fit-content !important;
7+
}
8+
9+
#container {
10+
display: grid;
11+
grid-template: 1fr / 1fr;
12+
place-content: center;
13+
}
14+
15+
#container.content ::slotted(*) {
16+
display: none;
17+
}
18+
19+
svg {
20+
fill: currentcolor;
21+
}
22+
123
:host([size=sm]) #container { --_size: var(--pf-global--icon--FontSize--sm, 10px); }
224
:host([size=md]) #container { --_size: var(--pf-global--icon--FontSize--md, 18px); }
325
:host([size=lg]) #container { --_size: var(--pf-global--icon--FontSize--lg, 24px); }

0 commit comments

Comments
 (0)