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

Listbox: Unexpected behaviour of Page Up/Down + infinite loop #14686

Closed
ramires83 opened this issue Feb 1, 2024 · 2 comments
Closed

Listbox: Unexpected behaviour of Page Up/Down + infinite loop #14686

ramires83 opened this issue Feb 1, 2024 · 2 comments

Comments

@ramires83
Copy link

Describe the bug

Page Up behaves as Page Down
Page Down key press causes infinite loop

Environment

SPA Angular web app
Windows environment

Reproducer

No response

Angular version

16.2.9

PrimeNG version

16.9.4-lts

Build / Runtime

Angular CLI App

Language

TypeScript

Node version (for AoT issues node --version)

18.17.0

Browser(s)

Chrome 121.0.6167.85 (Official Build) (64-bit)

Steps to reproduce the behavior

  1. Enable checkboxes (using checkbox=true), the filter input (filter=true) and the "Show all" option (showToggleAll=true).
  2. Click one of the items in the list to bring keyboard focus to the UL element (.p-listbox-list).
  3. Press the Page Down key.
    Expected behaviour: List scrolls down
    Actual behaviour: nothing happens
  4. Press the Page Up key.
    Expected behaviour: No action as the list is already scrolled up
    Actual behaviour: List scrolls down
  5. Press the Page Up key again.
    Expected behaviour: List scrolls up
    Actual behaviour: nothing happens
  6. Press the Page Down key again
    Expected behaviour: nothing happens as the list is already scrolled down
    Actual behaviour: Control enters an infinite loop and becomes unresponsive

Expected behavior

Page Down scrolls the list down
Page Up scrolls the list up
Control remains responsive

@ramires83 ramires83 added the Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible label Feb 1, 2024
@cetincakiroglu cetincakiroglu added this to the 17.7.0 milestone Feb 12, 2024
@cetincakiroglu cetincakiroglu added Status: Pending Review Issue or pull request is being reviewed by Core Team and removed Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible labels Feb 12, 2024
@cetincakiroglu cetincakiroglu modified the milestones: 17.7.0, 17.8.0 Feb 12, 2024
@cetincakiroglu cetincakiroglu self-assigned this Feb 22, 2024
@cetincakiroglu
Copy link
Contributor

Hi,

I've checked in both 17.7.0 and 16.9.5-lts versions and couldn't detect the issue. Could you please share a stackblitz example so we can identify the issue clearly?

See the examples:
v17.7.0: https://stackblitz.com/edit/dmauwb?file=src%2Fapp%2Fdemo%2Flistbox-basic-demo.html,src%2Fapp%2Fdemo%2Flistbox-basic-demo.ts

v16.9.5-lts: https://stackblitz.com/edit/rxs1db?file=src%2Fapp%2Fdemo%2Flistbox-virtual-scroll-demo.ts,src%2Fapp%2Fdemo%2Flistbox-virtual-scroll-demo.html

While waiting for the example, I'm moving the issue to the next milestone.

@cetincakiroglu cetincakiroglu added the Resolution: Needs More Information More information about the issue is needed to find a correct solution label Feb 22, 2024
@cetincakiroglu cetincakiroglu modified the milestones: 17.8.0, 17.9.0, 17.10.0 Feb 22, 2024
@cetincakiroglu cetincakiroglu modified the milestones: 17.10.0, 17.11.0 Mar 5, 2024
@cetincakiroglu
Copy link
Contributor

Closing the issue due to inactivity. Feel free to open a new issue with the reproducer.

@cetincakiroglu cetincakiroglu removed Status: Pending Review Issue or pull request is being reviewed by Core Team Resolution: Needs More Information More information about the issue is needed to find a correct solution labels Mar 13, 2024
@cetincakiroglu cetincakiroglu removed their assignment Mar 13, 2024
@cetincakiroglu cetincakiroglu removed this from the 17.11.0 milestone Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants