Skip to content

Conversation

@kmcfaul
Copy link
Contributor

@kmcfaul kmcfaul commented Sep 23, 2024

Closes #4169.

@patternfly-build
Copy link
Collaborator

patternfly-build commented Sep 23, 2024

@kmcfaul
Copy link
Contributor Author

kmcfaul commented Sep 23, 2024

@tlabaj The changes in ColorFamily (minus the isEditorial flag) and ColorSwatch were the changes I had to make to get the build to work with the updated prerelease versions

edit - update with version bump and fix went in another PR and got merged so should be set

Comment on lines 189 to 190
className="pf-v6-u-font-size-lg"
isEditorial
Copy link
Contributor

Choose a reason for hiding this comment

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

This element has the utility class className="pf-v6-u-font-size-lg", which will override isEditorial - want to try removing the utility class font-size and see if the isEditorial styling gives either the same or an appropriate font-size?

@lboehling
Copy link

clicked aroudn through several pages and from what i can tell this looks good from the design side!

Copy link
Contributor

@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.

👍👍

Copy link
Contributor

@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.

🐸👍

@nicolethoen nicolethoen merged commit 154cca6 into main Oct 1, 2024
@nicolethoen nicolethoen deleted the editorial-text branch January 2, 2025 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use long form text variation

6 participants