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.

Sign up for GitHub

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

Jump to bottom

minor ui improvement: focus some "default" fields #1974

Open
hydrargyrum opened this issue Apr 15, 2023 · 4 comments · Fixed by #1976
Open

minor ui improvement: focus some "default" fields #1974

hydrargyrum opened this issue Apr 15, 2023 · 4 comments · Fixed by #1976
Labels
accessibility accessibility & standards compliance enhancement ergonomy user experience template HTML rendering
Milestone
0.14.0

Comments

@hydrargyrum
Copy link

hydrargyrum commented Apr 15, 2023 •
edited
Loading

When landing on the login page (/login), the username field should be automatically focused, or at least be accessible with a single tab press instead of the 8th press.

Also, when landing on the "shaare" page (/admin/add-shaare), the URL field should be focused.

And when clicking on the "search" header icon (the magnifier), at least one of the 2 search fields should be focused (let's say the leftmost one: text search).
@hydrargyrum hydrargyrum changed the title minor ui improvement: focus login field minor ui improvement: focus some "default" fields Apr 15, 2023
ArthurHoaro added a commit to ArthurHoaro/Shaarli that referenced this issue Apr 15, 2023
@ArthurHoaro
Fix autofocus: load bulk action input on linklist only
70c03d4 
Bulk action form and input were loaded with autofocus feature on all pages, preventing other input field to benefit from autofocus feature.

Fixes shaarli#1974
@ArthurHoaro ArthurHoaro mentioned this issue Apr 15, 2023
Merged
@ArthurHoaro
Copy link
Member

Thanks for the report! This was actually the case and got broken by another feature recently.
I'm submitting a first quick fix for that.

I'm keeping this issue open because I like your suggestions:

  • autofocus search field in the dynamic header
  • review our "tab access" order for accessibility, especially for login page

@ArthurHoaro ArthurHoaro added template HTML rendering accessibility accessibility & standards compliance labels Apr 15, 2023
@ArthurHoaro ArthurHoaro added this to the 0.13.0 milestone Apr 15, 2023
@ArthurHoaro ArthurHoaro added the bug it's broken! label Apr 15, 2023
@ArthurHoaro ArthurHoaro reopened this Apr 15, 2023
@nodiscc nodiscc modified the milestones: 0.13.0, 0.14.0 Nov 22, 2023
@nodiscc nodiscc added enhancement ergonomy user experience and removed bug it's broken! labels Mar 26, 2024
@bttrx
Copy link

bttrx commented Sep 29, 2024

👍

@nodiscc
Copy link
Member

nodiscc commented Sep 29, 2024

@bttrx hi, please avoid 👍 -only comments as they notify everyone subscribed tot he issue or repository. Use the 👍 button in the first post, they are reflected here (or even better, send a patch ;) )

@bttrx
Copy link

bttrx commented Sep 30, 2024

send a patch

@nodiscc We'll see. No promise.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
accessibility accessibility & standards compliance enhancement ergonomy user experience template HTML rendering
Projects
None yet
Milestone
0.14.0
Development

Successfully merging a pull request may close this issue.

Fix autofocus: load bulk action input on linklist only ArthurHoaro/Shaarli
4 participants
@ArthurHoaro @hydrargyrum @nodiscc @bttrx