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

Focus global search input on open #41652

Merged
merged 1 commit into from
Nov 23, 2023
Merged

Focus global search input on open #41652

merged 1 commit into from
Nov 23, 2023

Conversation

nfebe
Copy link
Contributor

@nfebe nfebe commented Nov 21, 2023

For better accesibility, the search input in the global search modal should be focused when the search modal is opened.

Resolves: #41651

@solracsf solracsf added this to the Nextcloud 28 milestone Nov 21, 2023
@blizzz blizzz mentioned this pull request Nov 22, 2023
5 tasks
Copy link
Member

@marcoambrosini marcoambrosini left a comment

Choose a reason for hiding this comment

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

Tested and doesn't work for me.
Wouldn't it be better to watch isVisible and call focusInput whenever the value flips to true?

@blizzz blizzz modified the milestones: Nextcloud 28, Nextcloud 29 Nov 23, 2023
@nfebe nfebe force-pushed the global-search-focus-input branch from 097219b to 32230cc Compare November 23, 2023 11:41
@nfebe
Copy link
Contributor Author

nfebe commented Nov 23, 2023

Tested and doesn't work for me.
Wouldn't it be better to watch isVisible and call focusInput whenever the value flips to true?

Resolved @marcoambrosini

Copy link
Member

@marcoambrosini marcoambrosini left a comment

Choose a reason for hiding this comment

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

Works now :)

@nfebe nfebe force-pushed the global-search-focus-input branch from 32230cc to 55830a4 Compare November 23, 2023 16:06
@AndyScherzinger
Copy link
Member

/compile amend /

For better accesibility, the search input in the global search
modal should be focused when the search modal is opened.

Resolves: #41651

Signed-off-by: fenn-cs <[email protected]>
Signed-off-by: nextcloud-command <[email protected]>
@nextcloud-command nextcloud-command force-pushed the global-search-focus-input branch from 55830a4 to 8c6c0d1 Compare November 23, 2023 17:40
@blizzz
Copy link
Member

blizzz commented Nov 23, 2023

What about the failing cypress tests?

@AndyScherzinger
Copy link
Member

AndyScherzinger commented Nov 23, 2023

What about the failing cypress tests?

  • theming/navigation-bar-settings fail everywhere
  • settings/apps only here (I'll restart it)

none of them seem even remotely connected to this change-set

@blizzz blizzz merged commit 5f159d4 into master Nov 23, 2023
40 of 42 checks passed
@blizzz blizzz deleted the global-search-focus-input branch November 23, 2023 19:38
@blizzz
Copy link
Member

blizzz commented Nov 23, 2023

/backport to stable28

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Left over polishing for global search from
6 participants