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

ALLGEMEIN: Doppelklicks auf Action Buttons verhindern #115

Open
2 tasks
carlobeltrame opened this issue Feb 8, 2021 · 0 comments
Open
2 tasks

ALLGEMEIN: Doppelklicks auf Action Buttons verhindern #115

carlobeltrame opened this issue Feb 8, 2021 · 0 comments
Labels
bug Something isn't working core

Comments

@carlobeltrame
Copy link
Member

carlobeltrame commented Feb 8, 2021 •
edited
Loading

Es scheint möglich einen Doppelclick auf den Senden Button von TextMessages zu machen. In diesem Fall könnte das in sehr seltenen Fällen zum doppelten Versand von vielen SMS führen. Bei anderen Action Buttons in der Applikation sind die Auswirkungen unklar.

  • data-disable-with wurde bereits eingefügt und scheint nicht wirklich zu greifen, untersuchen wieso nicht.

Techspec

https://github.com/hitobito/hitobito/blob/master/app/views/messages/_actions_show.html.haml#L17

  • data-disable-with zum Laufen bringen
  • Bei allen Action Buttons generell data-disable-with aktivieren
@mtnstar mtnstar changed the title Messages/TextMessage: Doppelklicks auf "Senden"-Button verhindern MESSAGES: Doppelklicks auf SMS "Senden"-Button verhindern Feb 10, 2021
@carlobeltrame carlobeltrame added core bug Something isn't working and removed interest-die-mitte labels Feb 11, 2021
@carlobeltrame carlobeltrame changed the title MESSAGES: Doppelklicks auf SMS "Senden"-Button verhindern ALLGEMEIN: Doppelklicks auf Action Buttons verhindern Feb 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working core
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@carlobeltrame @chrusu