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

[BUG] Searching in package list with key presses does not focus item #3355

Closed
4 tasks done
tkohlmeier opened this issue Feb 25, 2025 · 4 comments · Fixed by #3360
Closed
4 tasks done

[BUG] Searching in package list with key presses does not focus item #3355

tkohlmeier opened this issue Feb 25, 2025 · 4 comments · Fixed by #3360
Assignees
Labels
bug Something isn't working needs-further-study
Milestone

Comments

@tkohlmeier
Copy link
Contributor

Please confirm these before moving forward

  • I have searched for my issue and have not found a work-in-progress/duplicate/resolved issue.
  • I have tested that this issue has not been fixed in the latest (beta or stable) release.
  • I have checked the FAQ section for solutions.
  • This issue is about a bug (if it is not, please use the correct template).

UniGetUI Version

3.1.6

Windows version, edition, and architecture

Windows 11 Pro 10.0.26100.3194

Describe your issue

When clicking on a pakage in a package list, one can select a package by entering the name. The package is correctly selected and scrolled into view, but it does not get keyboard focus. When pressing up/down/space/enter, the previously selected item receives the event.

Steps to reproduce the issue

  • go to installed packages
  • click on a package
  • press a key to select another package
  • press space

-> The previously clicked item gets checked instead of the one selected after pressing the key.

UniGetUI Log

not relevant

Package Managers Logs

not relevant

Relevant information

No response

Screenshots and videos

No response

@tkohlmeier tkohlmeier added the bug Something isn't working label Feb 25, 2025
@marticliment marticliment added this to the Bugs milestone Feb 25, 2025
@mrixner
Copy link
Contributor

mrixner commented Feb 25, 2025

I attempted to fix this when I originally implemented this functionality (see #3018), however I believe we determined that this is unfortunately not possible with the current UI toolkit's functionality. I would love if this would work (and as an aside, if the package list didn't reset the scroll position anytime the window loses focus), but was unable to find a way to do so.

@mrixner
Copy link
Contributor

mrixner commented Feb 25, 2025

and as an aside, if the package list didn't reset the scroll position anytime the window loses focus

Actually, I went to go open a bug report and I realized it's caused by the issue fixed by #3354

@mrixner
Copy link
Contributor

mrixner commented Feb 25, 2025

Upon further investigation, I found this StackOverflow article detailing a workaround. I will see if it works when I can.

@tkohlmeier
Copy link
Contributor Author

Yeah, I found that, too, but that is about a ListView, not an ItemsView. But I found another way to focus the package. It still does not work perfectly though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs-further-study
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants