-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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(files): move focus to sidebar on open #42444
fix(files): move focus to sidebar on open #42444
Conversation
8656a08
to
027868e
Compare
027868e
to
4ec2ba6
Compare
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.
Works great too! Thank you!
/compile / |
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.
Outside of a minor nitpick, LGTM! 🐘💙
This PR requires |
608e7d5
to
a70c3de
Compare
@@ -53,6 +53,7 @@ | |||
:close-after-click="!isMenu(action.id)" | |||
:data-cy-files-list-row-action="action.id" | |||
:is-menu="isMenu(action.id)" | |||
:aria-label="mountType === 'shared' && action.id === 'sharing-status' ? action.title([source]) : actionDisplayName(action)" |
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.
That seems really specific, where is this coming from and what does it have to do with that PR? :)
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.
From the next one line (title
prop) and the content of the button 7 lines below. I can move it to a new PR, but a button must have a label or text.
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.
@skjnldsv moved to another PR
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.
See comment
Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
/backport to stable28 |
a70c3de
to
17c9f73
Compare
|
/backport to stable28 |
Summary
See commits for details.
See nextcloud-libraries/nextcloud-vue#5219 for screenshots
Checklist