Opened 10 years ago
Closed 10 years ago
#29451 closed defect (bug) (fixed)
"customize your site"-button overlaps column
Reported by: | pixolin | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 4.1 | Priority: | normal |
Severity: | normal | Version: | 3.8 |
Component: | Administration | Keywords: | has-patch |
Focuses: | ui, administration | Cc: |
Description
Dashboard has a "Welcome to WordPress"-section that comes with a button "customize your site".
While this button perfectly fits in the column untranslated, depending on screen size it overlaps when translated e.g. into the much longer "Dein WordPress individualisieren". (An image that shows the overlapping can be found at https://imgur.com/I6AazLv)
To my understanding, this could be fixed by setting a max-width to the button's CSS class:
.wp-core-ui .button-group.button-hero .button, .wp-core-ui .button.button-hero { max-width: 100%; }
Attachments (5)
Change History (17)
#1
@
10 years ago
- Component changed from Formatting to Administration
- Focuses ui administration added; accessibility removed
#4
@
10 years ago
For German this is fixed now by changing the string on GlotPress.
But the CSS from @shooper looks fine and would be a more bulletproof version for this button.
#5
@
10 years ago
- Milestone changed from Awaiting Review to 4.1
29451.patch fixes the line height: 29451.png.
#6
@
10 years ago
- Owner set to SergeyBiryukov
- Resolution set to fixed
- Status changed from new to closed
In 29790:
#7
@
10 years ago
I don't know if this comment from Helen is relevant here, but maybe switch to em insteadt of pixel?
https://core.trac.wordpress.org/ticket/23189#comment:4
#8
@
10 years ago
- Resolution fixed deleted
- Status changed from closed to reopened
Yeah, I guess we should use em
for line-height
.
#9
@
10 years ago
- Keywords has-patch added
Changed to line-height to use em's instead of pixels, patch added.
#10
@
10 years ago
the button font size is 14px (not 13px)
the last patch sets a line-height of 1.538em
thus the computed line height value would be 21.532px (in Chrome dev tools inspector is 21.5319995880127px)
if you want a computed value of 20px, then the magic number is 1.4285714 (unitless is better)
Created a patch that gets this partially fixed. See: http://imgur.com/pVvH0vx
Line height is off when it wraps now. Not sure how to handle this. Over to someone else with better CSS chops :)