Skip to content

[Progress] Add Progress.Label and locale prop #1666

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 11 commits into from
Apr 16, 2025

Conversation

mj12albert
Copy link
Member

@mj12albert mj12albert commented Apr 2, 2025

Demo: https://deploy-preview-1666--base-ui.netlify.app/react/components/progress

Dropped the getAriaLabel prop as Progress.Label should be used to provide an accessible name

Also:

Closes #1201

Sorry, something went wrong.

@mj12albert mj12albert added the component: progress This is the name of the generic UI component, not the React module! label Apr 2, 2025
Copy link

netlify bot commented Apr 2, 2025

Deploy Preview for base-ui ready!

Name Link
🔨 Latest commit daed554
🔍 Latest deploy log https://app.netlify.com/sites/base-ui/deploys/67ff76cb73df420008208a9e
😎 Deploy Preview https://deploy-preview-1666--base-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@mj12albert mj12albert changed the title [Progress] Add Progress.Label [Progress] Add Progress.Label and locale prop Apr 2, 2025
@mj12albert mj12albert marked this pull request as ready for review April 2, 2025 15:01
@mj12albert mj12albert force-pushed the feat/progress-label branch from ccad4a5 to df6f1ca Compare April 8, 2025 08:14
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Apr 8, 2025
@mj12albert mj12albert force-pushed the feat/progress-label branch from df6f1ca to e1d106a Compare April 8, 2025 08:40
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Apr 8, 2025
@mj12albert mj12albert force-pushed the feat/progress-label branch from e1d106a to cf07145 Compare April 11, 2025 10:40
@mj12albert mj12albert force-pushed the feat/progress-label branch from cf07145 to 6ba0a11 Compare April 14, 2025 08:14
@mj12albert mj12albert force-pushed the feat/progress-label branch 2 times, most recently from 00001fb to 0feb51f Compare April 14, 2025 08:26
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Apr 15, 2025
@mj12albert mj12albert force-pushed the feat/progress-label branch from 0feb51f to e5ac57f Compare April 15, 2025 06:16
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Apr 15, 2025
@mj12albert mj12albert force-pushed the feat/progress-label branch from 1f6ee54 to 66f9344 Compare April 15, 2025 06:18
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Apr 15, 2025
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Apr 15, 2025
@mj12albert mj12albert merged commit f507d62 into mui:master Apr 16, 2025
22 checks passed
@mj12albert mj12albert deleted the feat/progress-label branch April 16, 2025 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change component: progress This is the name of the generic UI component, not the React module!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[progress] Styling feedback
2 participants