#53725 closed defect (bug) (fixed)
Classic Editor: 'new-post-slug' input missing a label
Reported by: |
|
Owned by: |
|
---|---|---|---|
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
@
23 months 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.
20 months ago
This ticket was mentioned in Slack in #accessibility by ryokuhi. View the logs.
19 months ago
This ticket was mentioned in Slack in #core-test by boniu91. View the logs.
19 months ago
This ticket was mentioned in Slack in #core-test by boniu91. View the logs.
19 months ago
This ticket was mentioned in Slack in #accessibility by ryokuhi. View the logs.
19 months ago
#9
@
19 months 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
@
19 months ago
- Owner set to audrasjb
- Status changed from new to reviewing
Self-assigning for proper testing using a screen-reader.
#11
@
19 months 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.
19 months ago
#13
Trac ticket: https://core.trac.wordpress.org/ticket/53725
#14
@
19 months 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.
19 months ago
hellofromtonya commented on PR #1922:
17 months 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.