-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[Feature Request]: Allow to hide next and previous buttons in PaginationNav #15013
Comments
Thank you for submitting a feature request. Your proposal is open and will soon be triaged by the Carbon team. |
Could you share more of the design and UX evidence to justify this change? My understanding of the initial development of this component is that leaving the element in the DOM is preferred so that there is consistency in the DOM for screenreader users. It supports the idea that screenreaders can read the button as "button (dimmed)" to convey the disabled status. For these reasons we probably won't modify this but I'm curious to know if you have any additional rationale. For instance, if we can show that removing the buttons would improve the screenreader experience, I think at that point we could reconsider the design. |
I will link you up with our design team, so they can explain you the reasoning and design specs behind this. I will contact you over Slack, since this will involve confidential information. |
I'm working on a component which adds 'pagination' to the 'Aside' area of the 'Leadspace with Foreground Asset' which is currently available in C4AEM. Initially, I built this component without the 'arrows' being displayed since the component is limited to 4 pages max and the 'Aside' area is tight on space. I've attended Carbon Design Office Hours and our initial conversation seemed to indicate that adding the ability to hide the 'Next' and 'Previous' arrows wouldn't be detrimental to accessibility since the amount of page is limited. I haven't contributed to Carbon yet, but I could look into this as an addition to the 'Pagination' component's options. |
The problem
We would like to migrate from our custom pagination component to PaginationNav. Our custom component hides next/previous button on the first/last page, instead of disabling it, like the current PaginationNav does.
The solution
Allow to hide next and previous buttons, somehow like this:
When the next/previous button is hidden, one more button linking to a page should by displayed. That way the width of the pagination will be the same on all pages. For example, the first page of pagination for
itemsShown: 4
andtotalItems: 9
currently looks like this:With
hideDisabledControls
set it would become:Examples
https://research.ibm.com/people
Application/PAL
https://research.ibm.com/
Business priority
Medium Priority = upcoming release but is not pressing
Available extra resources
No response
Code of Conduct
The text was updated successfully, but these errors were encountered: