search

gysmo38 / mitsubishi2MQTT

Mitsubishi to MQTT with ESP8266 module
GNU Lesser General Public License v2.1
393 stars 141 forks source link

Confirmation before reset configuration #54

Closed spfmoby closed 4 years ago

spfmoby commented 4 years ago

I know the reset configuration button is red, but a simple click by mistake and all is gone. Maybe adding a very basic javascript command on the button could be useful: something like a onclick="return confirm('Are you sure?')" or equivalent could prevent this.

What do you think about it?

shampeon commented 4 years ago

This is an easy fix, and yeah, we should definitely confirm before resetting. PR coming.

gysmo38 commented 4 years ago

Fixed thank @shampeon