search

0x0ece / yopa-native

0 stars 2 forks source link

Fix #103 feedback on submit password press #108

Closed jacksilv closed 6 years ago

jacksilv commented 6 years ago

Still perfectible, but at least there is a visual feedback. Comments are welcome :)

0x0ece commented 6 years ago

This is a great start. Also, please remember that if the user is in Armored mode and doesn't input the right passphrase, we silently fail without a message. I think we should also fix that, i.e. show a simple error message "Wrong master password [for Category]."

0x0ece commented 6 years ago

Sei un po' un lier... :) I pensavo più ad un messaggio inline nel prompt, che un secondo prompt. Cmq per me va bene direi, ma fai file di un task per fissarlo.

Nel mentre ho preso la mia e tua PR e le ho mandate online insieme.

jacksilv commented 6 years ago

Dicamo che si sarebbero stati modi più complicati :). Quando dici messaggio inline nel prompt, cosa intendi? Pes esempio una scritta che appare sotto il field text con scritto "Wrong....", o il field text che scompare completamente e viene sostituito dalla scritta di errore?

0x0ece commented 6 years ago

La prima, tipo validation error dell’input.

jacksilv commented 6 years ago

Ma se ci aggiungessimo una bella Vibration? https://facebook.github.io/react-native/docs/vibration.html

Ciao, Jacopo

Il giorno 21 nov 2017, alle ore 17:19, Emanuele Cesena notifications@github.com ha scritto:

La prima, tipo validation error dell’input

On Tue, Nov 21, 2017 at 00:14 jacksilv notifications@github.com wrote:

Dicamo che si sarebbero stati modi più complicati :). Quando dici messaggio inline nel prompt, cosa intendi? Pes esempio una scritta che appare sotto il field text con scritto "Wrong....", o il field text che scompare completamente e viene sostituito dalla scritta di errore?

Il giorno mar 21 nov 2017 alle ore 04:42 Emanuele Cesena < notifications@github.com> ha scritto:

Sei un po' un lier... :) I pensavo più ad un messaggio inline nel prompt, che un secondo prompt. Cmq per me va bene direi, ma fai file di un task per fissarlo.

Nel mentre ho preso la mia e tua PR e le ho mandate online insieme.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/0x0ece/yopa-native/pull/108#issuecomment-345907791, or mute the thread < https://github.com/notifications/unsubscribe-auth/ARuwZ3FvNkhaWb38enTlFiaBbP-KSh05ks5s4kaggaJpZM4QjZlo

.

— You are receiving this because your review was requested.

Reply to this email directly, view it on GitHub https://github.com/0x0ece/yopa-native/pull/108#issuecomment-345949041, or mute the thread https://github.com/notifications/unsubscribe-auth/ABbEGNW5ImwHpNqAyGJiVaTpFDcAuxFqks5s4oaBgaJpZM4QjZlo .

-- Emanuele Cesena emanuele.cesena@gmail.com http://www.theneeds.com

Il corpo non ha ideali — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

0x0ece commented 6 years ago

Mi sembra un filo estremo, ma lascio a te la scelta.