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

[BITV] 9.2.4.3/3.8 - When the "Select file to link to" dialog is closed, the focus is not returned to the "Link file" button but lost (beginning of page). (2) #4852

Closed
1 task
AndyScherzinger opened this issue Mar 1, 2023 · 5 comments
Assignees
Labels
Milestone

Comments

@Pytal

This comment was marked as outdated.

@AndyScherzinger
Copy link
Member Author

Please check again @Pytal - editor should now be available on master

@Pytal Pytal removed their assignment Oct 12, 2023
@susnux susnux self-assigned this Oct 12, 2023
@susnux susnux transferred this issue from nextcloud/server Oct 12, 2023
@susnux
Copy link
Contributor

susnux commented Oct 12, 2023

Problem:

  1. If you click in the menu bar on the link symbol
  2. Then on Link to File
  3. Then the Filepicker opens
  4. Then close the Filepicker without selecting anything

Now the menu is closed and the focus is on the begin of the document.

Instead the focus should stay on the open menu, on the "Link to file" action entry.

@juliusknorr
Copy link
Member

juliusknorr commented Oct 20, 2023

I can no longer reproduce this issue on latest master/main. @susnux Could you quickly confirm? Otherwise I'd be curious if a step is missing.

I noticed a different one that the viewer closes if i press Escape in the file picker. Looks like generally the escape key event is not stopped in NcModal. Proposed a fix for that in nextcloud-libraries/nextcloud-vue#4677

@juliusknorr juliusknorr self-assigned this Oct 20, 2023
@juliusknorr juliusknorr moved this to 🏗️ In progress in 📝 Office team Oct 20, 2023
@susnux
Copy link
Contributor

susnux commented Oct 21, 2023

It somewhat works now as the focus stays at least on the link menu toggle.

But I am not sure if that is enough as maybe the menu needs to stay open? But at least better 🎉

@susnux susnux closed this as completed Oct 21, 2023
@github-project-automation github-project-automation bot moved this from 🏗️ In progress to ☑️ Done in 📝 Office team Oct 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

No branches or pull requests

4 participants