owncloud / twofactor_totp

🔑 Second factor TOTP (Google Authenticator) provider for ownCloud
GNU Affero General Public License v3.0
9 stars 9 forks source link

Align the code input field with the submit button on login #249

Closed JammingBen closed 2 years ago

JammingBen commented 2 years ago

... to match with the core login layout.

Related: https://github.com/owncloud/twofactor_totp/issues/237

Before:

image

After:

image

CLAassistant commented 2 years ago

CLA assistant check
All committers have signed the CLA.

JammingBen commented 2 years ago

The warning text above still exceeds the width, but that's a general core thing. We could also reduce the warning's width, but sometimes the texts are quite large... that would result in many line breaks.

@JanAckermann What do you think?

ownclouders commented 2 years ago

:boom: Acceptance tests pipeline webUITwoFactTOTP-master-chrome-mariadb10.2-php7.4 failed. The build has been cancelled.

https://drone.owncloud.com/owncloud/twofactor_totp/1552/13/1

ownclouders commented 2 years ago

:boom: Acceptance tests pipeline webUITwoFactTOTP-master-chrome-mariadb10.2-php7.3 failed. The build has been cancelled.

https://drone.owncloud.com/owncloud/twofactor_totp/1552/12/1

sonarcloud[bot] commented 2 years ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information