Skip to content

Comments

[6.1] Display of translated translation title (workflow)#46806

Draft
heelc29 wants to merge 14 commits intojoomla:6.1-devfrom
heelc29:6.1/workflow/translation-translated-title
Draft

[6.1] Display of translated translation title (workflow)#46806
heelc29 wants to merge 14 commits intojoomla:6.1-devfrom
heelc29:6.1/workflow/translation-translated-title

Conversation

@heelc29
Copy link
Contributor

@heelc29 heelc29 commented Jan 31, 2026

This pull request builds on #46798, so that should be tested/merged first.

Summary of Changes

Since #35457 the transitions titles are translateable
For workflow and stage there is an extra field which shows the translated text and for transition it's missing.
image
This PRs adds the extra field with the translated value.
image

Testing Instructions

Edit an existing transition (you have to enable workflow integration)

Actual result BEFORE applying this Pull Request

only language key is there
image

Expected result AFTER applying this Pull Request

Translated value is displayed (here for example german)
image

Link to documentations

Please select:

  • No documentation changes for docs.joomla.org needed
  • No documentation changes for manual.joomla.org needed

@brianteeman
Copy link
Contributor

did you intend to include all the changes from your other pr with tests

@heelc29
Copy link
Contributor Author

heelc29 commented Jan 31, 2026

did you intend to include all the changes from your other pr with tests

Yes (also the reason why in draft state; and an upmerge from 5.4/6.0 missing)
image

and this PR activates the relevant test
image

@HLeithner
Copy link
Member

I personally would like to go into the other direction and removing the additional fields. We have a translation/override manager for this.

@brianteeman
Copy link
Contributor

@HLeithner that doesn't let you have a multilingual admin. This does.

@HLeithner
Copy link
Member

you will still have the translation in the overview, if you have 10 languages you have 10 lines, which is ... not so nice. (read the source code wrong) Alternative, show the translation as "inlinehelp" (always show) for example

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants