Skip to content

[4.1] Add text COM_WORKFLOW_STAGE_FORM_NEW to com_workflow.ini#37393

Merged
bembelimen merged 2 commits intojoomla:4.1-devfrom
richard67:4.1-dev-workflows-language-strings-fix
Mar 28, 2022
Merged

[4.1] Add text COM_WORKFLOW_STAGE_FORM_NEW to com_workflow.ini#37393
bembelimen merged 2 commits intojoomla:4.1-devfrom
richard67:4.1-dev-workflows-language-strings-fix

Conversation

@richard67
Copy link
Copy Markdown
Member

@richard67 richard67 commented Mar 28, 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.

@brianteeman
Copy link
Copy Markdown
Contributor

Can you delete the old string?

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

@richard67
Copy link
Copy Markdown
Member Author

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.

@brianteeman
Copy link
Copy Markdown
Contributor

See #32060

@richard67
Copy link
Copy Markdown
Member Author

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).

@richard67
Copy link
Copy Markdown
Member Author

Testing instructions updated. Please test.

@richard67 richard67 added the a11y Accessibility label Mar 28, 2022
@richard67 richard67 added this to the Joomla 4.1.1 milestone Mar 28, 2022
@brianteeman
Copy link
Copy Markdown
Contributor

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.

1 similar comment
@Quy
Copy link
Copy Markdown
Contributor

Quy commented Mar 28, 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.

@Quy Quy removed the a11y Accessibility label Mar 28, 2022
@joomla-cms-bot joomla-cms-bot removed this from the Joomla 4.1.1 milestone Mar 28, 2022
@Quy
Copy link
Copy Markdown
Contributor

Quy commented Mar 28, 2022

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Mar 28, 2022
@richard67 richard67 added this to the Joomla 4.1.1 milestone Mar 28, 2022
@richard67 richard67 added the a11y Accessibility label Mar 28, 2022
@bembelimen bembelimen merged commit dad35a5 into joomla:4.1-dev Mar 28, 2022
@bembelimen
Copy link
Copy Markdown
Contributor

Thx

@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Mar 28, 2022
@richard67 richard67 deleted the 4.1-dev-workflows-language-strings-fix branch March 29, 2022 06:20
Kostelano added a commit to JPathRu/localisation that referenced this pull request Mar 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a11y Accessibility Language Change This is for Translators

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants