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

[BITV]: Not possible to add already selected photos to the album via keyboard #2238

Closed
JuliaKirschenheuter opened this issue Jan 8, 2024 · 8 comments
Assignees
Labels
4. to release Ready to be released and/or waiting for tests to finish a11y28checked needed for a11y accessibility

Comments

@JuliaKirschenheuter
Copy link
Contributor

  1. Create an album
  2. Try to select some photos and add them to the album via keyboard

Peek 2024-01-08 18-05

It is not possible to reach "Add to ..." and "Add" buttons via keyboard

@JuliaKirschenheuter JuliaKirschenheuter added 1. to develop Accepted and waiting to be taken care of accessibility labels Jan 8, 2024
@Pytal Pytal self-assigned this Jan 10, 2024
@Pytal Pytal added 2. developing Work in progress and removed 1. to develop Accepted and waiting to be taken care of labels Jan 17, 2024
@Pytal

This comment was marked as resolved.

@Pytal Pytal removed their assignment Jan 24, 2024
@Pytal

This comment was marked as resolved.

@JuliaKirschenheuter
Copy link
Contributor Author

Thanks @Pytal!

@Pytal
Copy link
Member

Pytal commented Jan 27, 2024

This is a tricky bug as we use focus trap + virtual scrolling here. Focus trap finds the initial tabbable nodes but as you scroll the elements are rerendered and seems to cause the glitch. updateContainerElements does not seem to work, still not sure why though.

According to the focusin event the next node is correctly focused but immediately afterwards the focus is moved to body

Draft semi-working patch to bring back focus to the correct next element in Pytal/focus-trap#1

@Pytal
Copy link
Member

Pytal commented Jan 27, 2024

Regarding the virtual scrolling part, there is a new related bug with the virtual-scrolled files list where if you have many files and tab past the last file to the start of the page and then back into the list it will focus not on the first file but on the first currently rendered file i.e. 90th file row instead of the first

I believe there was some exception made for infinite scrolling before so maybe not an issue @JuliaKirschenheuter?

@ShGKme
Copy link
Contributor

ShGKme commented Jan 28, 2024

Can we disable focus-trap here and implement our own simple focus trap?

@Pytal
Copy link
Member

Pytal commented Jan 29, 2024

Can we disable focus-trap here and implement our own simple focus trap?

Just tried and seems to make it worse, tabbing at the glitch point jumps straight to the bottom instead of the next photo

@Pytal Pytal self-assigned this Jan 29, 2024
@Pytal Pytal added 4. to release Ready to be released and/or waiting for tests to finish and removed 2. developing Work in progress labels Jan 30, 2024
@JuliaKirschenheuter
Copy link
Contributor Author

close as done

@szaimen szaimen added the a11y28checked needed for a11y label Feb 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish a11y28checked needed for a11y accessibility
Projects
None yet
Development

No branches or pull requests

4 participants