Buttons / Arrows in TOTP

[MariusBluem]

Since nextcloud/server#1641 brought back the "Log In" button, I think we should overthink the buttons and arrows in TOTP too ... even if I am not a big fan of this decision I am a big fan of uniformity ... So ... Lets remove this arrow from TOTP and replace it with an dedicated button 😉

@jancborchardt @ChristophWurst @nextcloud/designers

[jancborchardt]

@MariusBluem can you add screenshots for clarification? :)

[ChristophWurst]

https://github.com/nextcloud/twofactor_totp/blob/master/screenshots/enter_challenge.png

[ChristophWurst]

Was changed with #48.

And we use that styling in the backup codes app too, which is part of NC11…

[My1]

uniformity aside, I honestly prefer buttons rather than that arrow, because it's more direct.

[Dennis1993]

Where I can see the change?
If I use Microsoft Edge, the arrow to submit the PIN is covered by the X from Edge itself.

The Edge is displaying in each input the X to remove the written text. Maybe we use a button or can remove the X with the following Code:

input[type=text]::-ms-clear {
    display: none;
}

Maybe it is fixed with #48 I don't know :D Then sorry for my post :)

[ChristophWurst]

Fixed by #149, forgot to close this ticket :-)