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
Fix key navigation on test details #5309
Conversation
By the way, I guess we have a ticket for that. However, I couldn't find it. (I came across this problem again when working on https://progress.opensuse.org/issues/136028.) |
This is poo#113468 I think? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems like a sensible fix!
10e1720
to
b7b54fb
Compare
@Mergifyio rebase |
✅ Nothing to do for rebase action |
@kalikiana as long as is not in rebasing won't help |
@Mergifyio rebase |
When viewing a test live and test module details are added to the page dynamically, then the key navigation breaks. It skips steps instead of just selecting the next/previous step. This is because the event handler is registered multiple times. This change fixes the issue by making sure the event handler is only registered once. Related ticket: https://progress.opensuse.org/issues/113468
✅ Branch has been successfully rebased |
b7b54fb
to
f1b115c
Compare
When viewing a test live and test module details are added to the page dynamically, then the key navigation breaks. It skips steps instead of just selecting the next/previous step. This is because the event handler is registered multiple times. This change fixes the issue by making sure the event handler is only registered once.