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

[cds-pagination]: pagination right side raises both cds-select-selected and cds-pagination-changed-current events #11923

Closed
2 tasks done
AlessandroPomponio opened this issue Jul 12, 2024 · 0 comments
Labels
bug Something isn't working dev Needs some dev work

Comments

@AlessandroPomponio
Copy link

Description

When interacting with the right side of the pagination component I expect only cds-pagination-changed-current to be changed, as per the comments in : https://github.com/carbon-design-system/carbon-for-ibm-dotcom/blob/main/packages/carbon-web-components/src/components/pagination/pagination.ts#L456-L468

As it can be seen from the following video, however, both events are raised, meaning that we end up setting the elements per page to a value based on the selected page:

2024-07-12.15-46-05.mp4

For completeness, our definition of the cds-pagination component in VueJS is:

        <cds-pagination
          :page-size="10"
          :total-items="history.tags.length + history.untagged.length"
          @cds-select-selected="handleTablePageSizesSelectChanged"
          @cds-pagination-changed-current="handleTablePaginationChangedCurrent"
        >
          <cds-select-item
            v-for="(option, optionIdx) in tablePaginationPageSizeOptions"
            :value="option"
            >{{ option }}</cds-select-item
          >
        </cds-pagination>

Component(s) impacted

cds-pagination

Browser

Microsoft Edge

Carbon for IBM.com version

2.10

Severity

Severity 2 = Aspects of design is broken, and impedes users in a significant way, but there is a way to complete their tasks. Affects major functionality, has a workaround.

Application/website

ST4SD Registry UI

Package

@carbon/web-components

CodeSandbox example

N/A

Steps to reproduce the issue (if applicable)

No response

Release date (if applicable)

No response

Code of Conduct

@AlessandroPomponio AlessandroPomponio added bug Something isn't working dev Needs some dev work labels Jul 12, 2024
@andy-blum andy-blum closed this as not planned Won't fix, can't repro, duplicate, stale Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working dev Needs some dev work
Projects
Status: Done
Development

No branches or pull requests

2 participants