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

search input: set focus on search input by default #178

Merged

Conversation

AoNoOokami
Copy link

@AoNoOokami AoNoOokami commented May 5, 2020

Co-Authored-by: Alicia Zangger alicia.zangger@rero.ch

Why are you opening this PR?

To implement task 1462: https://tree.taiga.io/project/rero21-reroils/task/1462?kanban-status=1224894
To close rero/rero-ils#928

How to test?

Go to different brief views of professional interface.
Check that the focus is automatically set on the search input.
Go to the document editor.
Check that there is no conflict with the focus in the editor.

Code review check list

  • Commit message template compliance.
  • Commit message without typos.
  • File names.
  • Functions names.
  • Functions docstrings.
  • Unnecessary commited files?
  • Extracted translations?

* Removes 'autofocus' html attribute.
* Sets the focus on search input.
* Closes rero/rero-ils#928.

Co-Authored-by: Alicia Zangger <alicia.zangger@rero.ch>
@AoNoOokami AoNoOokami force-pushed the zaa-#1462-search-input-focus-all-views branch from 54aa997 to 6e53b0e Compare May 5, 2020 12:14
@AoNoOokami AoNoOokami self-assigned this May 5, 2020
@AoNoOokami AoNoOokami requested review from jma and sebdeleze May 5, 2020 15:50
@AoNoOokami AoNoOokami requested a review from benerken May 6, 2020 14:06
@AoNoOokami AoNoOokami removed the request for review from jma May 8, 2020 12:05
@AoNoOokami AoNoOokami merged commit 08db504 into rero:dev May 8, 2020
@AoNoOokami AoNoOokami deleted the zaa-#1462-search-input-focus-all-views branch June 2, 2020 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Focus not set in many views
3 participants