Opened 9 years ago
Closed 9 years ago
#34944 closed defect (bug) (fixed)
The "required" title attributes are not translatable in wp-admin/includes/media.php
Reported by: | andg | Owned by: | afercia |
---|---|---|---|
Milestone: | 4.5 | Priority: | normal |
Severity: | normal | Version: | 4.4 |
Component: | I18N | Keywords: | has-patch has-screenshots commit |
Focuses: | accessibility, administration | Cc: |
Description
In the wp-admin/includes/media.php
file, a few title
attributes with an hardcoded value of required
(the title is present because the content of those links is just an asterisk) are not translatable.
Attachments (4)
Change History (19)
#1
@
9 years ago
- Keywords has-patch added
- Milestone changed from Awaiting Review to 4.5
Looks good to me. Perhaps strings need context for translators?
#2
@
9 years ago
Something along the line of input fields title attribute
, perhaps? Better have context than not, preparing another patch. Thanks for the feedback, @swissspidy!
@
9 years ago
Make the "required" title attributes translatable in wp-admin/includes/media.php, with proper context.
This ticket was mentioned in Slack in #accessibility by afercia. View the logs.
9 years ago
#6
in reply to:
↑ 4
;
follow-up:
↓ 7
@
9 years ago
Replying to afercia:
Was struggling a bit trying to find where these title attributes are actually used. Looks like some of them were used in the pre-3.5.0 media upload popup, not sure if still used somewhere, maybe worth checking a bit before introducing new strings?
The old media UI can be re-enabled with a plugin, and I've seen some plugins using it.
Should we perhaps add aria-label
in addition to (or instead of) the title attribute?
#7
in reply to:
↑ 6
@
9 years ago
- Focuses accessibility added
Replying to SergeyBiryukov:
The old media UI can be re-enabled with a plugin, and I've seen some plugins using it.
Then the best thing would probably be mimicking the Comment form and:
- remove the title attribute
- add the sentence "Required fields are marked *" before the form
- move the asterisk inside the labels
- add both
aria-required="true"
andrequired
on the required fields
#8
@
9 years ago
- Keywords has-screenshots added
First pass with the new approach. Any feedback and improvements welcome :)
#9
@
9 years ago
Screenshot with an example of the patch applied. The asterisk won't be replaced by the icons but the "no", "OK" and spinner icons will still work:
#10
follow-up:
↓ 12
@
9 years ago
- Keywords close added
- Owner ocean90 deleted
I'm voting for a wontfix. The old media UI is … old and we shouldn't encourage people to still using it.
#11
@
9 years ago
I can see both points. I do think we should fix it. Not to encourage people but to create a better experience if a user is still seeing this. Wasn't this also not enabled if a certain action/filter was used?
#12
in reply to:
↑ 10
@
9 years ago
Replying to ocean90:
we shouldn't encourage people to still using it.
I see both points but if won't fix, then maybe it should be deprecated? I'd vote to just fix it, it's simpler :)
#13
follow-up:
↓ 14
@
9 years ago
- Owner set to afercia
I don't see this as a priority, but also don't have a problem with it being fixed. Up to you.
#14
in reply to:
↑ 13
@
9 years ago
- Keywords commit added; close removed
Replying to mikeschroder:
I don't see this as a priority, but also don't have a problem with it being fixed. Up to you.
Refreshed patch. Definitely not a priority, I'd just like to kill all the title attributes :)
Make the "required" title attributes translatable in wp-admin/includes/media.php