Skip to content

fix(pagination): Use pageText instead of itemsPerPageText to fix a11y label#2270

Merged
zvonimirfras merged 3 commits intocarbon-design-system:masterfrom
exequiel09:fix-pagination-page-selector-translation-key-#2264
Oct 5, 2022
Merged

fix(pagination): Use pageText instead of itemsPerPageText to fix a11y label#2270
zvonimirfras merged 3 commits intocarbon-design-system:masterfrom
exequiel09:fix-pagination-page-selector-translation-key-#2264

Conversation

@exequiel09
Copy link
Copy Markdown
Contributor

closes #2264

Update pagination component to fix a11y label of page selector

Changelog

New

N/A

Changed

Change hidden label for page selector

Removed

N/A

@exequiel09 exequiel09 requested a review from a team as a code owner October 2, 2022 09:15
@netlify
Copy link
Copy Markdown

netlify bot commented Oct 2, 2022

Deploy Preview for carbon-components-angular ready!

Name Link
🔨 Latest commit aee9729
🔍 Latest deploy log https://app.netlify.com/sites/carbon-components-angular/deploys/633d9367238fe70008bc07b0
😎 Deploy Preview https://deploy-preview-2270--carbon-components-angular.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 settings.

@Akshat55
Copy link
Copy Markdown
Contributor

Akshat55 commented Oct 3, 2022

Thanks for contributing @exequiel09! Could you please follow these instructions & sign off on the commit:

To add your Signed-off-by line to every commit in this branch:

  1. Ensure you have a local copy of your branch by checking out the pull request locally via command line.
  2. In your local branch, run: git rebase HEAD~1 --signoff
  3. Force push your changes to overwrite the branch: git push --force-with-lease origin fix-pagination-page-selector-translation-key-#2264

… label

Signed-off-by: Exequiel Ceasar Navarrete <dev@exequielnavarrete.me>
@exequiel09 exequiel09 force-pushed the fix-pagination-page-selector-translation-key-#2264 branch from b3e4647 to 46e8c1c Compare October 4, 2022 00:07
@exequiel09
Copy link
Copy Markdown
Contributor Author

@Akshat55 commit sign-off done

Copy link
Copy Markdown
Contributor

@Akshat55 Akshat55 left a comment

Choose a reason for hiding this comment

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

Approved!

@zvonimirfras zvonimirfras enabled auto-merge October 5, 2022 13:57
@zvonimirfras zvonimirfras merged commit f05eaaa into carbon-design-system:master Oct 5, 2022
@carbon-bot
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 4.61.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 5.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@exequiel09 exequiel09 deleted the fix-pagination-page-selector-translation-key-#2264 branch April 26, 2024 12:42
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.

[Pagination] Page selector using ITEMS_PER_PAGE translation key

4 participants