Skip to content
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

fix(pagination): prevent text from disappearing at certain sizes #4776

Merged

Conversation

emyarod
Copy link
Member

@emyarod emyarod commented Nov 26, 2019

Closes #4248

ref #1518

Pagination text was being hidden in a media query style block which is also triggered by zooming in on the page. This behavior does not seem to match the component spec

Changelog

Changed

  • move component padding style rules out of media query block

Removed

  • display: none on pagination component text

Testing / Reviewing

Ensure the component matches the spec and the original zoom/reflow issue is no longer present

@emyarod emyarod added this to the ♿Carbon WCAG Compliance♿ milestone Nov 26, 2019
@emyarod emyarod requested a review from a team as a code owner November 26, 2019 07:46
@emyarod emyarod added this to Pull requests 🙏🏿 in Carbon React accessibility via automation Nov 26, 2019
@ghost ghost requested review from abbeyhrt and asudoh November 26, 2019 07:46
@netlify
Copy link

netlify bot commented Nov 26, 2019

Deploy preview for the-carbon-components ready!

Built with commit 485d67f

https://deploy-preview-4776--the-carbon-components.netlify.com

@netlify
Copy link

netlify bot commented Nov 26, 2019

Deploy preview for carbon-elements ready!

Built with commit 485d67f

https://deploy-preview-4776--carbon-elements.netlify.com

@netlify
Copy link

netlify bot commented Nov 26, 2019

Deploy preview for carbon-components-react ready!

Built with commit 485d67f

https://deploy-preview-4776--carbon-components-react.netlify.com

Copy link
Member

@laurenmrice laurenmrice left a comment

Choose a reason for hiding this comment

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

looks good to me !

Copy link
Contributor

@asudoh asudoh left a comment

Choose a reason for hiding this comment

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

LGTM 👍 - Thanks @emyarod!

@asudoh asudoh merged commit debec0b into carbon-design-system:master Nov 27, 2019
Carbon React accessibility automation moved this from Pull requests 🙏🏿 to Closed issues 🏁 Nov 27, 2019
@emyarod emyarod deleted the 4248-pagination-text-display branch November 27, 2019 08:59
@mattrosno mattrosno removed this from the ♿Carbon WCAG Compliance♿ milestone Dec 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Carbon React accessibility
Closed issues 🏁
Development

Successfully merging this pull request may close these issues.

AVT 2 - Pagination has reflow issues over
4 participants