Skip to content

fix(files): make sure we focus the parent actions submenu entry - #41338

Merged
skjnldsv merged 2 commits into
masterfrom
fix/nested-actions-back-focus
Jan 31, 2024
Merged

fix(files): make sure we focus the parent actions submenu entry#41338
skjnldsv merged 2 commits into
masterfrom
fix/nested-actions-back-focus

Conversation

@skjnldsv

Copy link
Copy Markdown
Member

Comment threadapps/files/src/components/FileEntry/FileEntryActions.vue
@susnux

Copy link
Copy Markdown
Contributor

I like this solution :)

@blizzzblizzz mentioned this pull request Nov 9, 2023

@emoral435emoral435 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@emoral435

Copy link
Copy Markdown
Contributor

/compile amend /

This was referenced Nov 10, 2023
@blizzzblizzz mentioned this pull request Nov 20, 2023
5 tasks
@blizzzblizzz modified the milestones: Nextcloud 28, Nextcloud 29Nov 23, 2023
@skjnldsv
skjnldsvforce-pushed the fix/nested-actions-back-focus branch from 23c98e1 to 7d1b719CompareJanuary 31, 2024 12:20
@skjnldsv

Copy link
Copy Markdown
MemberAuthor

/compile

skjnldsvand others added 2 commits January 31, 2024 17:07
Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@skjnldsv
skjnldsvforce-pushed the fix/nested-actions-back-focus branch from cd90639 to 8257cbaCompareJanuary 31, 2024 16:08
@skjnldsvskjnldsv removed the 3. to review Waiting for reviews label Jan 31, 2024
@skjnldsvskjnldsv added the 4. to release Ready to be released and/or waiting for tests to finish label Jan 31, 2024
@skjnldsv

Copy link
Copy Markdown
MemberAuthor

/backport 23899de to sable28

@skjnldsv
skjnldsv merged commit 02e9cd6 into masterJan 31, 2024
@skjnldsv
skjnldsv deleted the fix/nested-actions-back-focus branch January 31, 2024 17:06
@backportbot

This comment was marked as resolved.

@skjnldsv

Copy link
Copy Markdown
MemberAuthor

/backport 23899de to stable28

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finishaccessibilityfeature: files

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants

@skjnldsv@susnux@emoral435@blizzz@nextcloud-command