#7080 closed enhancement (fixed)
Fix Layout in wp-admin User > Extended Profile screen
Reported by: | mercime | Owned by: | mercime |
---|---|---|---|
Milestone: | 2.6 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Extended Profile | Keywords: | commit reporter-feedback |
Cc: |
Description
Markup improvements in #6678 for the front end - notably using fieldsets for checkbox, datebox, and radio buttons fields - has affected the styles in our w-admin User > Extended Profiles screen in the back end.
In addition, the datebox, the screen reader label
s for date, month, and year are showing up because class="bp-screen-reader-text" is hardcoded in in class-bp-xprofile-field-type-datebox.php
To do:
- Add style support for the new
legend
element in the Extended Profiles screen. - Add
is_admin()
conditional inside theclass
attribute to renderscreen-reader-text
value for WP admin screen and renderbp-screen-reader-text
for the front-end Profile screen.
Patch and screenshots coming up.
Attachments (5)
Change History (13)
#1
@
9 years ago
- Keywords commit reporter-feedback added
- Milestone changed from Awaiting Review to Under Consideration
This is great. Thanks for being on top of this.
How does this look for older versions of WordPress once we have the newer markup & styling support?
#2
@
9 years ago
@johnjamesjacoby :) I applied the patch to test install - BP Trunk / WP 4.0.11 - and layout is fixed there as well. I need to increase the spacing between the "clear" link below the radio input fields and will upload revised patch for that in a little while.
Also just noticed that I will need to improve styles for this screen's mobile view, yikes. Will create a new ticket for this.
#3
@
9 years ago
Also just noticed that I will need to improve styles for this screen's mobile view, yikes
Mobile UI/UX on this page has never been great, so we are okay to make incremental improvements here.
Keep up the great work!
#5
@
9 years ago
- Owner set to mercime
- Resolution set to fixed
- Status changed from new to closed
In 10799:
Screen before patch