#53725 closed defect (bug) (fixed)
Classic Editor: 'new-post-slug' input missing a label
Reported by: | sabernhardt | Owned by: | audrasjb |
---|---|---|---|
Milestone: | 5.9 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Editor | Keywords: | has-patch commit |
Focuses: | accessibility, coding-standards | Cc: |
Description
The input for changing the permalink of a post does not have a label in the Classic Editor.
Attachments (2)
Change History (20)
#2
@
3 years ago
The change looks good to me, thanks @sabernhardt!
I can confirm the URL Slug
string already exists and is translated in fr_FR (https://translate.wordpress.org/projects/wp/dev/fr/default/?filters%5Bstatus%5D=either&filters%5Boriginal_id%5D=7578448&filters%5Btranslation_id%5D=62892203) so I'm not sure why it's not translated. Maybe because the existing occurence comes from the block editor?
This ticket was mentioned in Slack in #core by sabernhardt. View the logs.
3 years ago
This ticket was mentioned in Slack in #accessibility by ryokuhi. View the logs.
3 years ago
This ticket was mentioned in Slack in #core-test by boniu91. View the logs.
3 years ago
This ticket was mentioned in Slack in #core-test by boniu91. View the logs.
3 years ago
This ticket was mentioned in Slack in #accessibility by ryokuhi. View the logs.
3 years ago
#9
@
3 years ago
Note that the proposed label is not visual, so it would need testing with a screen reader.
Testing the extra part to make sure the browser does not run spellcheck on the input (when enabled) might be easier visibly, so you can verify the jagged underline no longer shows under an intentionally misspelled word.
#10
@
3 years ago
- Owner set to audrasjb
- Status changed from new to reviewing
Self-assigning for proper testing using a screen-reader.
#11
@
3 years ago
I tested on a visual browser to make sure it doesn't trigger spellcheck on the input field, and it looks good to me. Also, the label is correctly rendered at least on VoiceOver.
This ticket was mentioned in PR #1922 on WordPress/wordpress-develop by audrasjb.
3 years ago
#13
Trac ticket: https://core.trac.wordpress.org/ticket/53725
#14
@
3 years ago
Just added a GitHub PR to run checks, and it appears slug_label
needs to be defined. Fixing this right now :)
This ticket was mentioned in Slack in #core-test by monikarao. View the logs.
3 years ago
hellofromtonya commented on PR #1922:
3 years ago
#18
Committed via https://core.trac.wordpress.org/changeset/52225.
The patch includes a
spellcheck
attribute, which was suggested on #50499.Also, I think the translation needs something else. I chose the existing string "URL Slug" instead of new text, and I still have English there with a French profile.