#42157 closed defect (bug) (fixed)
Labels are misaligned for radio and checkbox controls in customizer
Reported by: | sayedwp | Owned by: | westonruter |
---|---|---|---|
Milestone: | 4.9 | Priority: | normal |
Severity: | normal | Version: | 4.9 |
Component: | Customize | Keywords: | has-patch has-screenshots commit |
Focuses: | ui | Cc: |
Attachments (11)
Change History (28)
#1
@
7 years ago
- Keywords good-first-bug needs-patch added
- Milestone changed from Awaiting Review to 4.9
This ticket was mentioned in Slack in #core by shitalmarakana. View the logs.
7 years ago
#4
@
7 years ago
@shitalmarakana I have tested your patch in Chrome and Firefox and it looks good. All radio buttons seem to be aligned properly inside customizer now. Thank you!
#5
@
7 years ago
- Summary changed from Labels are misaligned for radio controls in customizer to Labels are misaligned for radio and checkbox controls in customizer
@Shital Patel I think updating the label
in forms.css file would affect other labels in WordPress specially in post edit and general options page as I can imagin. You might want to be a bit more specific and add this css in
wp-admin/css/customize-controls.css
file.
.customize-control-radio label, .customize-control-checkbox label { vertical-align: top; }
I just noticed checkboxe labels like "Display Site Title and Tagline" in "Site Identity" section is also not aligned which would be fixed with the above css.
But then again it would affect other customizer controls like "Menu Locations" checkboxes, so those would need to be fixed as well.
#7
@
7 years ago
@sayedwp Not sure if this works but I have tested my patch in different device widths (320 to 1366) and they seem to be OK. Also if this works, the same rule needs to be added in custom-constrols-rtl.css as well. Patch follows.
Please let me know what you think.
Thank you.
#8
@
7 years ago
@Shital Patel Thanks that's almost perfect, I have just made some improvements in your patch.
@subrataemfluence Sorry, I am not aware of rtl support in customizer.
@westonruter I found one more css issue while checking this one in small screen ( was also there in 4.8 ) which I have covered in my patch.
#9
@
7 years ago
- Keywords good-first-bug removed
- Owner set to melchoyce
- Status changed from new to reviewing
#10
@
7 years ago
@sayedwp @subrataemfluence RTL CSS will be generated automatically as part of the build. No need to supply any patch for RTL files.
#11
@
7 years ago
Looks good to me on Firefox and Chrome 👍
Can I get some help testing Safari and IE11+?
#13
@
7 years ago
- Keywords commit added
- Owner changed from melchoyce to westonruter
- Status changed from reviewing to accepted
This is probably a regression introduced in #33085 via [41740] as the
label
element no longer wraps the radio input.