-
Notifications
You must be signed in to change notification settings - Fork 156
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
feat: add focus trap to left sidebar #5027
Conversation
💥 Acceptance tests Move failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15140/
|
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.
Doesn't work in Firefox (stuck at the close button) and Safari (not at all) though
Both are issues already on master and not due to the focus trap - I would see this is a different PR to keep the concerns separated. |
Also, pls note that this PR doesn't bring any focus management regarding focusing the first element in the sidebar when opened... |
Agreed. We should also update the nav to collapse on route change on mobile, not sure if this has been documented somewhere already :) |
b366c0d
to
4b4a410
Compare
💥 Acceptance tests Move failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15142/
|
a76d6fb
to
25225b1
Compare
💥 Acceptance tests webUISharingInternalUsersRoot2 failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15148/
|
💥 Acceptance tests Move failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15148/
|
💥 Acceptance tests webUIFiles1 failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15148/
|
💥 Acceptance tests webUISharingInternalUsersRoot1 failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15148/
|
💥 Acceptance tests webUIFiles2 failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15148/
|
💥 Acceptance tests webUISharingInternalUsers failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15148/
|
cc92b31
to
8fd6fa9
Compare
25225b1
to
0f17279
Compare
@kulmann focus trap activated only when the sidebar is an overlay now... It's still a bit useless though because of the missing focus trigger as mentioned here #5027 (comment) not sure yet how to handle this though as all the content is hidden inside the component 🤷 |
💥 Acceptance tests TrashbinFilesFolders failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15284/
|
💥 Acceptance tests TrashbinDelete failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15284/
|
💥 Acceptance tests MarkdownEditor failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15284/
|
💥 Acceptance tests SharingPublicDifferentRoles failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15284/
|
💥 Acceptance tests SharingPublicExpire failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15284/
|
💥 Acceptance tests webUISharingInternalUsersRoot2 failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15284/
|
💥 Acceptance tests SharingInternalGroups failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15284/
|
💥 Acceptance tests MarkdownEditor failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15348/
|
💥 Acceptance tests webUIDelete failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15348/
|
💥 Acceptance tests Favorites failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15348/
|
💥 Acceptance tests SharingInternalUsersExpireToRoot failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15348/
|
💥 Acceptance tests SharingInternalGroups failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15348/
|
💥 Acceptance tests webUISharingInternalUsersRoot1 failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15348/
|
💥 Acceptance tests SharingInternalGroupsRoot failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15348/
|
💥 Acceptance tests SharingInternalUsersExpire failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15348/
|
💥 Acceptance tests Upload failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15348/
|
💥 Acceptance tests Move failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15348/
|
💥 Acceptance tests SharingPermissionsUsers failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15348/
|
💥 Acceptance tests webUICreate failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15348/
|
💥 Acceptance tests SharingAutocompletion failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15348/
|
💥 Acceptance tests SharingPublicManagement failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15348/
|
💥 Acceptance tests SharingPermissionToRoot failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15348/
|
💥 Acceptance tests SharingPublicDifferentRoles failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15348/
|
💥 Acceptance tests ResharingToRoot failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15348/
|
💥 Acceptance tests webUISharingBasic failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15348/
|
💥 Acceptance tests webUISharingInternalUsersRoot2 failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15348/
|
💥 Acceptance tests webUIResharing failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15348/
|
💥 Acceptance tests webUIFiles1 failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15348/
|
💥 Acceptance tests webUIFiles2 failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15348/
|
💥 Acceptance tests SharingPublicBasic failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15348/
|
💥 Acceptance tests webUIBasic failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15348/
|
💥 Acceptance tests webUISharingInternalUsers failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15348/
|
💥 Acceptance tests webUIRename failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15348/
|
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.
LGTM 🎯
Description
We've added a focus trap to the left sidebar on smaller resolutions when it's collapsible.
If the sidebar is opened and focus, the focus stays within the sidebar.
How Has This Been Tested?
Types of changes