search

lumapu / ahoy

Various tools, examples, and documentation for communicating with Hoymiles microinverters
https://ahoydtu.de
Other
945 stars 221 forks source link

Passwortfelder mit "Augensymbol" um Passwörter temporär anzuzeigen #1669

Open DanielR92 opened 2 months ago

DanielR92 commented 2 months ago

Wird realisiert. Rückfrage, soll nur das aktuelle Feld auf den Toggle reagieren? Oder auch andere Passwort-Felder in Zukunft?

Welche Felder sind zu behandeln?

rmayergfx commented 2 months ago

Aus meiner Sicht wäre eine funktionierende Sicherheitsabfrage mit Passwortwiederholung wesentlich sinnvoller als ein "Auge" um das eingegebene Passwort anzusehen. Das Auge darf aus Sicherheitsgründen auch nur temporär angezeigt werden, d.h. nach dem Speichern müssten auf jeden Fall alle Werte gelöscht sein, damit niemand nach einem Reboot darauf zugreifen kann.

Aktuell gibt man das Kennwort einmal ein und drückt auf speichern, dumm wenn man sich vertippt hat.

lumapu commented 2 months ago

die Felder enthalten auch jetzt schon einen Platzhalter und nicht das 'echte' Passwort. Gibt es diese Funktion noch nicht als html-Feld?

DanielR92 commented 2 months ago

Eine Passwortwiederholung ist schnell eingebaut. Das mit dem "Auge" ist ja klar. Da kann man einiges noch einbauen. Gern dürfen auch andere sich dazu gesellen. :)

lumapu commented 2 months ago

finde die checkbox ganz gut, ist quick'n'dirty. Wir könnten auch per CSS versuchen die Checkbox in dieses Auge zu wandeln und hinten ans Feld anzuhängen - habe dafür mal nach "bootstrap password eye" gesucht, ein bisschen was findet man da. Mir wäre es ganz lieb, wenn dieses Feature nicht den Rahmen des ESPs sprenge würde, also KISS 😉