-
Notifications
You must be signed in to change notification settings - Fork 2
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
[Feature Request]: Suggestion to improve Accessibility #43
Comments
Done changes for 1&2 in b6b60f5 Test especially with both prompt visible |
Tested 1&2 and LGTM Edit: 4 is also fixed probably by nr 2. 2nd Edit: i think i have every part of the UI listed now, tried to be thorough with it |
Is there any existing prompt that would refocus on previously focused element on dismiss? |
Hmm idk will test later today! Also @ChunkyProgrammer does these suggestions even make sense from an accessibility standpoint? I mean to me it does and i think it works like this in the browser (am not sure about that though) |
@PikachuEXE its very inconsistent how things are handled rn In the privacy settings menu click Clear search cache or remove watch history -> use tab and press enter on no to cancel the prompt -> press tab again and see that the hamburger menu is on focus just like u saw in some of the screenrecordings In the privacy settings menu click Remove all subscriptions / profiles -> tab to no and press enter -> press tab and see that focus is on the privacy settings tab Not sure if this is the same because it opens a OS prompt for importing/exporting but in the data settings when u click on one of the options -> and cancel that prompt the button where u initially clicked on is still on focus. There is on exclusion to this and that is the export Subscriptions setting because that does bring up a FT prompt. When u cancel that prompt and hit tab again the Data settings tab is in focus |
Let's move non playlist related stuff into another issue (probably in main repo) |
(3) should be fixed in a624a85 |
The reason is that the prompt element is rooted in that section, when prompt is closed = destroyed, the next focus target would still be picked from the last parent (I guess) But this is not true with "global" prompt where the prompt element is rooted in "app" To be able to auto focus on last focused element after closing a "global" prompt is another thing to do but should not be part of the playlist feature implementation |
Can confirm 3 is fixed |
if we cut out non related playlist stuff then i think we should only do 5/6/11/12 |
Done a change in e2b1f4f But it doesn't work on invisible element (e.g. the button only visible on hover/activate) |
tested 5/6/11/12 and all works i'd like to suggest an better improvement on 6 though, now i focuses back on the create playlist button when u just created the playlist. I think its better to have the focus on the just created playlist. |
I tried calling
But it's visible again with tab |
Hmm okay then i would like to focus it on the seachbar so i can tab to that just added playlist, because maybe that person added that playlist but dont want the video to be added in there so in that case they can still deselect the playlist |
Done in 349c363 |
LGTM! Edit: all accessibility issues related to playlists are solved. I'll keep this open so i can migrate this when playlists are implemented. |
Updated |
Tested and looks like everything in this issue is addressed except for 4.1 but idk if something can be done about that |
It's possible but might be a bit complicated |
ok i'll leave it open so i know i need to migrate this over later |
(10) should re-tested |
10 lgtm! |
4.1 partially fixed by having focus shifted to current prompt |
LGTM, closing this issue. |
Guidelines
Problem Description
VirtualBoxVM_0nwSRWQvjt.mp4
VirtualBoxVM_BqTKBnVsIu.mp4
VirtualBoxVM_WzMhO1XeHq.mp4
VirtualBoxVM_Z31xlmMF3W.mp4
4.1. Expansion on 4, i can cancel the prompt where i need to select a playlist to add my videos to but i can still create a playlist
VirtualBoxVM_sXedP6lFFO.mp4
VirtualBoxVM_75fdSj2Ems.mp4
VirtualBoxVM_UhhtWcJi4n.mp4
VirtualBoxVM_NCP90pYTrl.mp4
add-button-playlist-video-page.mp4
add-button-video-player.mp4
playlist-add-video.mp4
del-playlist.mp4
rem-watched.mp4
cancel-save-copy-playlistt.mp4
save-playlist-prompt.mp4
cancel-playlist-prompt.mp4
Proposed Solution
Alternatives Considered
None.
Issue Labels
ease of use improvement
Additional Information
No response
The text was updated successfully, but these errors were encountered: