#7080 closed enhancement (fixed)
Fix Layout in wp-admin User > Extended Profile screen
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| 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 labels 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
legendelement in the Extended Profiles screen. - Add
is_admin()conditional inside theclassattribute to renderscreen-reader-textvalue for WP admin screen and renderbp-screen-reader-textfor the front-end Profile screen.
Patch and screenshots coming up.
Attachments (5)
Change History (13)
#1
@
10 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
@
10 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
@
10 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
@
10 years ago
- Owner set to mercime
- Resolution set to fixed
- Status changed from new to closed
In 10799:
Screen before patch