Skip to content
This repository has been archived by the owner on Nov 21, 2020. It is now read-only.

Use modal for action's message #151

Open
rlecellier opened this issue Sep 20, 2016 · 7 comments
Open

Use modal for action's message #151

rlecellier opened this issue Sep 20, 2016 · 7 comments
Milestone

Comments

@rlecellier
Copy link
Contributor

rlecellier commented Sep 20, 2016

With included react-bootstrap we can easly replace alert by pretty modale.
For example 'delete field' and 'reset form' in the builder.

@almet
Copy link
Member

almet commented Sep 21, 2016

I actually had in mind to replace the links with an "are you sure" text, letting the user click again without having to move the cursor.

@rlecellier
Copy link
Contributor Author

good idea for the delete field.
about the reset form i think a modal still better.

@almet
Copy link
Member

almet commented Sep 21, 2016

Let's summon ux People on this topic: @Natouille @QuentinRoy ?

@almet
Copy link
Member

almet commented Sep 21, 2016

(personally the solution you are mentionning makes sense to me!)

@Natouille
Copy link

Delete field is a good idea (to be sure you don't delete teh whole form).
The alert box is OK to me.
About teh reset your form, it should appear but it should be a light design (your primary action is Save your form)

@rlecellier
Copy link
Contributor Author

@Natouille @almet please confirm that i'll well understand.

  • replace "delete field" confirmation from alert box to a second click.
  • do not replace the "reset form" alert box by a prettier css modal.
  • change "reset form" button by a lighter design as it's a secondary action.

I don't undestand why changing a hugly alert box by a pretty modal isn't a good thing.

@almet
Copy link
Member

almet commented Oct 20, 2016

Almost:

  • replace "delete field" confirmation from alert box to a second click.
  • do replace the "reset form" alert box by a prettier css modal.
  • change "reset form" button by a lighter design as it's a secondary action.

@almet almet added this to the v2 milestone Oct 20, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants