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] Code Toggle cannot be tabbed to or adjusted with Keyboard #116

Closed
liamie opened this issue Mar 26, 2020 · 13 comments
Closed

[BUG] Code Toggle cannot be tabbed to or adjusted with Keyboard #116

liamie opened this issue Mar 26, 2020 · 13 comments
Assignees
Labels
bug Something isn't working fixed Fixed and ready for testing released Code changes released tested System Test passed
Milestone

Comments

@liamie
Copy link
Contributor

liamie commented Mar 26, 2020

Tabbing seems to skip by this element in general and even after using it with mouse doesn't seem to be functional with keyboard

image

Additional context
This keyboard functionality is a requirement for Accessibility

@liamie liamie added the bug Something isn't working label Mar 26, 2020
@liamie
Copy link
Contributor Author

liamie commented Mar 26, 2020

Similar for official statistics toggle in build
image

@liamie
Copy link
Contributor Author

liamie commented Mar 27, 2020

Similar in release workflow
image

@liamie
Copy link
Contributor Author

liamie commented Apr 7, 2020

also seems to lead to ambiguity and obfuscation when trying to print this type of toggle
image

@liamie
Copy link
Contributor Author

liamie commented Aug 5, 2020

These toggles are prolific in the charts section also. This makes keyboard tabbing for options in charts impossible, mouse only access for options.
image

@lorenzobruni
Copy link
Member

@liamie
The fix to this issue has been already posted into the original GitHub project.
I have asked to commit and issue a new release for this fix... let's see.
minhur/bootstrap-toggle#125

If not committed shortly we will have to fork the project ourself.
We stay tuned.
Cheers

@lorenzobruni
Copy link
Member

I have forked the https://github.com/CSOIreland/bootstrap-toggle repository to implement this fix.

The tootles are now working with tab-keys and they can also be toggles with space/enter keys.
Can you please retest?

Thanks

@lorenzobruni lorenzobruni removed their assignment Oct 9, 2020
@lorenzobruni lorenzobruni added the fixed Fixed and ready for testing label Oct 9, 2020
@lorenzobruni lorenzobruni added this to the 2.6.0 Beta milestone Oct 9, 2020
@lorenzobruni
Copy link
Member

FYI @liamie

@liamie
Copy link
Contributor Author

liamie commented Oct 9, 2020

This issue is mostly fixed now.
Tabbing can pick up the toggle option and space can change the toggle value.
Printing displays the option chosen.
However currently Analytical choice produces a success Modal which disrupts the focus. Can the focus be returned to the toggle on close of the success modal?
Also can there be a distinctive highlight around the toggle when selected it's currently hard to tell when you are on it.

@liamie liamie assigned damienderoistecso and unassigned liamie Oct 9, 2020
@lorenzobruni
Copy link
Member

Also can there be a distinctive highlight around the toggle when selected it's currently hard to tell when you are on it.

This is there already... but the button is surrounded by the toggle frame, hence is not very visible... an example below.
image
image

However currently Analytical choice produces a success Modal which disrupts the focus. Can the focus be returned to the toggle on close of the success modal?

Sorry I didn't get this one... can you attach a screenshot or rephrase? Thanks

@liamie
Copy link
Contributor Author

liamie commented Oct 12, 2020

Also can there be a distinctive highlight around the toggle when selected it's currently hard to tell when you are on it.

This is there already... but the button is surrounded by the toggle frame, hence is not very visible... an example below.
image
image

Ok if the toggle cant be made more distinctive, like group below in the images, because of nesting in the frame can the text of the toggle be emphasised or something similar to provide a visual cue.
This is group selected not toggle
image
This is toggle selected
image

However currently Analytical choice produces a success Modal which disrupts the focus. Can the focus be returned to the toggle on close of the success modal?

Sorry I didn't get this one... can you attach a screenshot or rephrase? Thanks
When Analytical toggle selection is switched
image
appears but when this is closed the focus doesn't return to the toggle to proceed from there.
This may be separate issue unrelated to toggle specifically of returning focus to where dialog screens are launched from once closed unless there is an action resulting from the choice/close in the dialog.

@lorenzobruni
Copy link
Member

@liamie , thanks!
I agree we should address how the focus look because is almost imperceptible.
I will look into the return of the focus...

@damienderoistecso,
can you please inspect the CSS have have a look to how we can improve the visual effect of a focused toggle?
Thanks

@damienderoistecso
Copy link
Collaborator

@lorenzobruni possible issue with forked library. When we toggle now, error in console

image

@lorenzobruni
Copy link
Member

I have issued a new release of the bootstrap-toggle with both the console error fixed and the focus/blur enhanced.
The slider changes color then on focus/blur.
image
image

Unfortunately the success modal has no callback function (currently) so we cannot bring the focus back to anything.
Perhaps something to consider in the future?
Do you want to create an enhancement under the Client API repo to implement a callback function for the api.modal.success ?

Please test and let me know if the 2 items above are fixed.

Thanks!

@liamie liamie added the tested System Test passed label Oct 14, 2020
@liamie liamie assigned lorenzobruni and unassigned liamie Oct 14, 2020
@lorenzobruni lorenzobruni added the released Code changes released label Oct 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed Fixed and ready for testing released Code changes released tested System Test passed
Projects
None yet
Development

No branches or pull requests

3 participants