Opened 4 years ago
Closed 3 years ago
#52834 closed defect (bug) (fixed)
Reset password screen: improve buttons layout for better i18n
Reported by: | audrasjb | Owned by: | peterwilsoncc |
---|---|---|---|
Milestone: | 5.7.1 | Priority: | normal |
Severity: | normal | Version: | 5.7 |
Component: | Login and Registration | Keywords: | has-screenshots fixed-major has-patch commit dev-reviewed |
Focuses: | ui, css | Cc: |
Description
Related: #39638
Props to @pixelverbieger for pointing this out.
In some locales, the two buttons position is not ideal. They should ideally be stacked when there is not enough place leave them float.
Attachments (5)
Change History (22)
#2
@
4 years ago
- Keywords commit added; 2nd-opinion removed
Tested across a few languages of different lengths, looks good across all.
#3
@
3 years ago
Thank you @audrasjb for reporting this. I just noticed the same issue.
I hope we can get this in in a minor version.
#4
@
3 years ago
- Milestone changed from Awaiting Review to 5.7.1
I think this would be good to put in the upcoming minor release.
#5
@
3 years ago
- Owner set to peterwilsoncc
- Resolution set to fixed
- Status changed from new to closed
In 50569:
#6
@
3 years ago
- Keywords fixed-major dev-feedback added
- Resolution fixed deleted
- Status changed from closed to reopened
Reopening for commit to the 5.7 branch following second a committer's sign-off.
#7
@
3 years ago
- Keywords dev-reviewed added; dev-feedback removed
Based on my earlier review this is good to backport.
#9
@
3 years ago
- Resolution fixed deleted
- Status changed from closed to reopened
Login page button issue after removed the below CSS in #50572
.login .button-primary { float: right; }
I'm using a development version (5.8-alpha-50571).
#10
@
3 years ago
- Keywords needs-patch added; has-patch commit removed
I can reproduce the issue as well, thanks @mukesh27
#13
@
3 years ago
The above patch restore the previous rule and keep the new one used in password reset form.
German translation i18n issue