8000 fix(card): clean up utility classes in demos by srambach · Pull Request #6497 · patternfly/patternfly · GitHub
[go: up one dir, main page]

Skip to content

fix(card): clean up utility classes in demos#6497

Merged
mcoker merged 1 commit intopatternfly:v6from
srambach:6496-update-card-demo-icons-timestamp
Apr 1, 2024
Merged

fix(card): clean up utility classes in demos#6497
mcoker merged 1 commit intopatternfly:v6from
srambach:6496-update-card-demo-icons-timestamp

Conversation

@srambach
Copy link
Member
@srambach srambach commented Apr 1, 2024

Replaces some utility class use with the timestamp component and icon component. Also removes an un-supported light font weight.

Fixes #6496

@patternfly-build
Copy link
Collaborator
patternfly-build commented Apr 1, 2024

<i class="fas fa-exclamation-circle pf-v6-u-icon-color-status-success" aria-hidden="true"></i>
{{#> icon}}
{{#> icon-content icon-content--modifier="pf-m-success"}}
<i class="fas fa-check-circle" aria-hidden="true"></i>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch on the wrong icons being used here, especially this one!

Copy link
Contributor
@mcoker < 8000 a class="author Link--primary text-bold css-overflow-wrap-anywhere " show_full_name="false" data-hovercard-type="user" data-hovercard-url="/users/mcoker/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="/mcoker">mcoker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

L🎃TM!

@mcoker mcoker merged commit 18db650 into patternfly:v6 Apr 1, 2024
@patternfly-build
Copy link
Collaborator

🎉 This PR is included in version 6.0.0-alpha.115 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

0