#Dismissable Alerts

5 messages · Page 1 of 1 (latest)

safe ridge
#

Allow the user to dismiss certain alerts.

i.e. in my case, the alerts get in the way of the navigation button so I have to wait a few seconds for all the alerts to clear before I can start navigation. Primarily the two alerts are the dev-c3 alert and the nnlc enabled alert. They aren't critical alerts, so being able to dismiss them would be a good idea.

I'm looking at the alert code and may be able to implement this myself, so I'll see what I can do!

dusty shard
#

If you can raise a PR to us (or even to comma) that would be great!. Although I am not comfortable with making the dev-c3 alert dismissable, the other ones feel like ok to me. (orange ones I am not sure is what I mean)

#

We intentionally have the alert so people understand what's at stake when they are using a branch that's under development tho

safe ridge
#

It may be a PR to comma, since the code changes would be in the master branch, unless we can safely make changes to master.

The dev-c3 alert is understandable, but I think the NNLC alert could be dismissable.

dusty shard
#

Yeah, I like the concept of dismissable alerts tho, I've had cases where I would also would've love to dismiss something