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

Remove submit button from bottom right corner #24

Open
bastiion opened this issue Apr 2, 2022 · 1 comment
Open

Remove submit button from bottom right corner #24

bastiion opened this issue Apr 2, 2022 · 1 comment
Assignees
Labels
medium prio We want solve it in one of next sprints, but not before urgent tasks are done.

Comments

@bastiion
Copy link
Collaborator

bastiion commented Apr 2, 2022

From a ui point of perspective putting the send button directly at the same place where next button was, is not good. It could lead to accidentally submited forms

@johannesloetzsch
Copy link
Member

What's the problem with submitting accidentally?
Changing the form and submitting again is still possible.

I would argue that 1 user not submitting is much worse than 100 users submitting more often than needed.
For this reason, I planed that the temporary state should even be submitted when changing the wizard step: https://github.com/MissionLifeline/afg/blob/main/doc/src/architecture/user-session/sequence.md

In my opinion it is only important that the submission via the bottom right corner is not using the ?submit=finally option.
But let's discuss the idea of https://github.com/MissionLifeline/afg/blob/main/doc/src/architecture/user-session/summary.md next week…

@bastiion bastiion added the medium prio We want solve it in one of next sprints, but not before urgent tasks are done. label May 3, 2022
@bastiion bastiion self-assigned this May 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
medium prio We want solve it in one of next sprints, but not before urgent tasks are done.
Projects
None yet
Development

No branches or pull requests

2 participants