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

Avoid scrolling to enter a price #186

Open
CharlesNepote opened this issue Jan 22, 2024 · 2 comments · Fixed by #238
Open

Avoid scrolling to enter a price #186

CharlesNepote opened this issue Jan 22, 2024 · 2 comments · Fixed by #238

Comments

@CharlesNepote
Copy link
Member

Screenshot_20240122-134731_Firefox.jpg

@alexfauquette
Copy link
Member

If I understand correctly, a solution could be to move the focus to the price input when the code is accepted

@CharlesNepote
Copy link
Member Author

CharlesNepote commented Jan 26, 2024

If I understand correctly, a solution could be to move the focus to the price input when the code is accepted

It could be that way. Or you could try to have a simpler screen. There are many informations on this screen.

  • "Ajouter un seul prix" is quite big, it could be smaller.
  • "Information du prod..." is useless IMHO.
  • "[Scanner un code-barres] Taper un code-barres" could be simplified: "[Scanner] ou saisir"
    • there is no need to repeat "Code-barres" which is already written on previous line
  • "Détail du prix" is useless IMHO.
  • "Prix" is useless IMHO, already written inside the field.
  • "Preuve\n[Prendre une photo] Sélectionner dans votre galerie" could be simplified by "Photo de preuve\n[Photographier] ou Sélectionner" or "Gallerie" for the latter.
  • "Où et quand ?" is useless. I would display "Emplacement du magasin" and "Date" in the previous block.
  • "Date" title is useless (already written in the field).

@raphodn raphodn linked a pull request Jan 26, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Backlog
Development

Successfully merging a pull request may close this issue.

3 participants