? Language Change Release Blocker a11y ? Pending

User tests: Successful: Unsuccessful:

avatar richard67
richard67
28 Mar 2022

Pull Request for Issue #37390 part 1 .

Summary of Changes

Added the missing language string as discussed in the issue and removed the old one since it is not used anywhere directly or by string concatenation of the key.

Testing Instructions

Edit a workflow stage.

In the edit view, use the action "Save & New".

After that the aria-label of the form should have changed from "Edit Stage" to "Add Stage".

Actual result BEFORE applying this Pull Request

Aria label "COM_WORKFLOW_STAGE_FORM_ADD".

Expected result AFTER applying this Pull Request

Aria label "Add Stage".

Documentation Changes Required

None.

avatar richard67 richard67 - open - 28 Mar 2022
avatar richard67 richard67 - change - 28 Mar 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 28 Mar 2022
Category Administration Language & Strings
avatar brianteeman
brianteeman - comment - 28 Mar 2022

Can you delete the old string?

On my phone so can't check if it's used elsewhere

avatar richard67 richard67 - change - 28 Mar 2022
Labels Added: Language Change Release Blocker ?
avatar richard67
richard67 - comment - 28 Mar 2022

Can you delete the old string?

On my phone so can't check if it's used elsewhere

@brianteeman Done. As far as I could see it is not used anywhere else directly or for building it by concatenation.

But I still don't find where the string has to appear, i.e. which element shall have that aria-label.

avatar brianteeman
brianteeman - comment - 28 Mar 2022

See #32060

avatar richard67
richard67 - comment - 28 Mar 2022

See #32060

@brianteeman I saw the issue but the description there is not clear. Meanwhile I found it: You have to be in the edit view of a stage and then use the action "Save & New". After that the aria-label of the form changes from "Edit Stage" to the untranslated text (when the PR is not applied).

avatar richard67 richard67 - change - 28 Mar 2022
The description was changed
avatar richard67 richard67 - edited - 28 Mar 2022
avatar richard67
richard67 - comment - 28 Mar 2022

Testing instructions updated. Please test.

avatar richard67 richard67 - change - 28 Mar 2022
The description was changed
avatar richard67 richard67 - edited - 28 Mar 2022
avatar richard67 richard67 - change - 28 Mar 2022
The description was changed
avatar richard67 richard67 - edited - 28 Mar 2022
avatar brianteeman
brianteeman - comment - 28 Mar 2022

I have tested this item successfully on 7f86366


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37393.

avatar brianteeman brianteeman - test_item - 28 Mar 2022 - Tested successfully
avatar Quy
Quy - comment - 28 Mar 2022

I have tested this item successfully on 7f86366


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37393.

avatar Quy Quy - test_item - 28 Mar 2022 - Tested successfully
avatar Quy Quy - change - 28 Mar 2022
Status Pending Ready to Commit
avatar Quy
Quy - comment - 28 Mar 2022

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37393.

avatar bembelimen bembelimen - change - 28 Mar 2022
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-03-28 23:21:51
Closed_By bembelimen
Labels Added: ? a11y
avatar bembelimen bembelimen - close - 28 Mar 2022
avatar bembelimen bembelimen - merge - 28 Mar 2022
avatar bembelimen
bembelimen - comment - 28 Mar 2022

Thx

Add a Comment

Login with GitHub to post a comment