8000 feat(progress): display progress measure as tabular numbers by srambach · Pull Request #6985 · patternfly/patternfly · GitHub
[go: up one dir, main page]

Skip to content

feat(progress): display progress measure as tabular numbers#6985

Merged
mcoker merged 1 commit intopatternfly:v6from
srambach:5476-tabular-numbers-progress
Aug 20, 2024
Merged

feat(progress): display progress measure as tabular numbers#6985
mcoker merged 1 commit intopatternfly:v6from
srambach:5476-tabular-numbers-progress

Conversation

@srambach
Copy link
Member

This should take care of the only issue not addressed in another issue on #5476

This PR changes the progress measure to be tabular numbers.

@patternfly-build
Copy link
Collaborator
patternfly-build commented Aug 16, 2024

@srambach
Copy link
Member Author

@lboehling @andrew-ronaldson Just want to call out that this modifies all the "measure" numbers shown on a progress bar. If you'd prefer that it only affects those with static width measure or some other subset, please let me know.

@mcoker
Copy link
Contributor
mcoker commented Aug 16, 2024

Tabular numbers will go in these places:

When inside the progress bar
Screenshot 2024-08-16 at 3 23 37 PM

The "Step 2..." text
Screenshot 2024-08-16 at 3 24 34 PM

33% text
Screenshot 2024-08-16 at 3 23 51 PM

"2 of 5" text
Screenshot 2024-08-16 at 3 24 10 PM

33%
Screenshot 2024-08-16 at 3 36 25 PM

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.

LGTM as long as the places that were updated are where design intended 👍

@srambach srambach linked an issue Aug 19, 2024 that may be closed by this pull request
Copy link
Collaborator
@andrew-ronaldson andrew-ronaldson left a comment

Choose a reason for hiding this comment

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

Approved!

@mcoker mcoker merged commit ecf9e9e into patternfly:v6 Aug 20, 2024
@patternfly-build
Copy link
Collaborator

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

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.

Add Tabular Font to Dynamic Components

4 participants

0