Opened 10 months ago
Closed 9 months ago
#63006 closed defect (bug) (fixed)
Invalid aria-describedby for email if user not current user
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 6.8 | Priority: | normal |
| Severity: | normal | Version: | |
| Component: | Users | Keywords: | has-patch has-screenshots commit |
| Focuses: | accessibility | Cc: |
Description
https://github.com/WordPress/wordpress-develop/blob/trunk/src/wp-admin/user-edit.php#L548
<?php <input type="email" name="email" id="email" aria-describedby="email-description" value="<?php echo esc_attr( $profile_user->user_email ); ?>" class="regular-text ltr" /> <?php if ( $profile_user->ID === $current_user->ID ) : ?> <p class="description" id="email-description">
if user not current user, the aria-describedby is invalid and shouldn't be output
Attachments (12)
Change History (26)
#1
@
10 months ago
- Keywords needs-patch added
- Milestone changed from Awaiting Review to 6.8
- Severity changed from trivial to normal
This ticket was mentioned in PR #8376 on WordPress/wordpress-develop by @audrasjb.
10 months ago
#3
- Keywords has-patch added; needs-patch removed
@audrasjb commented on PR #8376:
10 months ago
#4
#6
@
9 months ago
Test Report
Before
https://app.screencast.com/bIKIj8rp25vRW
After
https://app.screencast.com/uphnvUBmHBdJ9
Environment
- OS: macOS
- Localhost Server: Nginx
- PHP: 8.3.0
- WordPress: 6.7.2
- Browser: Chrome
- Theme: TT4
Actual Results
- ✅ The patch is working fine and now the
aria-describedbyis not visible when the user is not a current user.
#7
@
9 months ago
Test Report
Patch Tested: https://github.com/WordPress/wordpress-develop/pull/8376
Environment:
WordPress - 6.8-beta2
OS - Windows
Browser - Firefox
Theme: Twenty Twenty Five
PHP - 8.2.12
Active Plugin: Beta Tester
Actual Results:
- Issue Resolved With Patch.✅
aria-describedbyis not visible when the user is not a current user.
Screenshots:
- Added Attachment
#8
@
9 months ago
Hello,
It's my first contribution during the Whodunit's contribution day.
I tested the patch and it's ok for me.
Best,
#10
@
9 months ago
Hi all,
Same as Nandor, during our Contribution Day at Whodunit, I tested this patch and it looks to work fine for me.
Have a great day!
#11
@
9 months ago
Hi all,
Same as Nandor, Marine and Quentin, during our Contribution Day at Whodunit, I tested this patch and it looks to work fine for me.
Have a great day!
#12
@
9 months ago
Test Report
Patch Tested: https://github.com/WordPress/wordpress-develop/pull/8376
Environment:
WordPress - 6.8-alpha-20250222.085844
Tested with playground.wordpress.net
Results:
aria-describedby is not added on other user edition and the description doesn't exist.
Screenshots:
Added Attachment
Good catch! We should indeed use the same condition to determine wether the attribute is relevant or not.