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

Notify user that changes are not persisted until form is sent #923

Open
2 tasks
amiedes opened this issue Sep 6, 2017 · 4 comments
Open
2 tasks

Notify user that changes are not persisted until form is sent #923

amiedes opened this issue Sep 6, 2017 · 4 comments

Comments

@amiedes
Copy link
Contributor

amiedes commented Sep 6, 2017

For nested forms that make use of modals to edit a subset of the information, we should make clear to the user that changes made inside the modal won't be saved until the whole form is sent (click "Update").

The "save" button on the modal only persists the changes on the DOM, not in the DB.

captura de pantalla 2017-09-06 a las 17 16 13

Affected screens:

  • Create / edit of processes
  • Create / edit of polls

Tasks:

@furilo
Copy link
Member

furilo commented Sep 26, 2017

@Crashillo can you do a proposal as how you would do this? My idea was to show a flash like message saying something like: "These changes will be saved only when you Update the process"

@furilo furilo assigned Crashillo and unassigned furilo Sep 26, 2017
@Crashillo
Copy link
Member

Definitely, I'd use an alert banner. Perhaps, with a different color, even more highlighted. That banner would come up when modal is saved. Below is the easiest implementation:
imagen

Due to the form height, it would be better if that banner was in the update sticky panel, as:
imagen

Another option is to place the banner as shown in the first image, but make it sticky all over the page.

As well, we could inform in the modified form field itself:
imagen
Not exclusively here, but inside the sticky also, to inform user where is/are the modifications.

Tell me your thoughts.

@amiedes
Copy link
Contributor Author

amiedes commented Sep 27, 2017

  • I like the sticky idea, since when a few questions have been added the banner won't be visible unless we do this.
  • In case we make it sticky, I'd put it in the update panel, otherwise maybe it's too "obtrusive"
  • I also think this will be really useful:

image

@furilo
Copy link
Member

furilo commented Sep 28, 2017

Let's go with the flash inside the sticky panel.

But avoid the icon for the time being (until we start using them in other places as well), and maybe make the font size the same as in Borrador / Publico ?

@Crashillo Crashillo removed their assignment Nov 15, 2017
@amiedes amiedes removed their assignment Mar 1, 2019
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

No branches or pull requests

3 participants