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

enhancement : don't block the ui when an operation is succeed #25

Closed
fabriceci opened this issue Aug 7, 2016 · 8 comments
Closed

enhancement : don't block the ui when an operation is succeed #25

fabriceci opened this issue Aug 7, 2016 · 8 comments

Comments

@fabriceci
Copy link
Contributor

fabriceci commented Aug 7, 2016

When you successfully do something (delete a file, add a file) there is a success message box that blocks the ui and force to click on the OK button (and just after the fm refresh the ui).

It will be better to have instead an instant refresh and a notification that disappears automatically after x seconds without blocking anything. (something like that).

@psolom
Copy link
Owner

psolom commented Aug 7, 2016

Noty is a nice plugin. It's ok for me, but I believe we have to force user to click OK button for some important messages (errors e.g.) to be sure he got the message. Anyway, this enhancement should take the lowest priority.

@simogeo
Copy link
Contributor

simogeo commented Aug 12, 2016

@fabriceci : set showConfirmation to false in config file.

@psolom
Copy link
Owner

psolom commented Aug 12, 2016

If I got @fabriceci right, he wants to display notifications with the help of showConfirmation, but to make them less obtrusive. They are not supposed to block the page and dissapear after N seconds. The Noty plugin is intended for a such behavior. But I doubt whether this is an appropriate behavior for an important messages, like errors etc. What do you think @simogeo ?

@simogeo
Copy link
Contributor

simogeo commented Aug 12, 2016

First, I wanted to be sure @fabriceci knew about showConfirmation option.

Noty is nice but does not handle prompting user input (which is needed and now handled with impromptu library). Both functionalities are useful and indeed, it would be great to have confirmation messages less intrusive ... but as you said before, it does not appear to me as a priority. A switch to a library handling both prompt dialog box and notifications would be perfect. According to me, alertify.js is more appropriate but is no more maintained ...

Right now, it would be useful to set showConfirmation to false on the default config file.

@jlaustill
Copy link
Contributor

Sweet Alert is pretty awesome for blocking alerts and dialogs. I've been using it and love it. But I have been looking for a nonblocking alerting library for awhile with no luck. I would like to find something that slides in alerts on the bottom right and then fades them out like facebook does, so I'll be following this conversation to see what y'all come up with :)

@simogeo
Copy link
Contributor

simogeo commented Aug 12, 2016

Didn't know Sweet alert ... Nice.
Anyone to maintain http://fabien-d.github.io/alertify.js ??? ;-))

@psolom
Copy link
Owner

psolom commented Aug 31, 2016

New notifications have implemented based on alertify plugin with my modifications:
https://github.com/servocoder/alertify.js

@simogeo @fabriceci @jlaustill
Check dev branch and #41

@simogeo
Copy link
Contributor

simogeo commented Aug 31, 2016

excellent ! great

@psolom psolom closed this as completed Sep 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants