Skip to content

Conversation

@brianteeman
Copy link
Contributor

The wrong string was being used in the dropdown to select a module position

Actual result BEFORE applying this Pull Request

image

Expected result AFTER applying this Pull Request

image

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

The wrong string was being used in the dropdown to select a module position
@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-4.2-dev labels Feb 24, 2023
@sandewt
Copy link
Contributor

sandewt commented Feb 24, 2023

Also @todo on other places:

JGLOBAL_TYPE_OR_SELECT_CATEGORY="Type or Select a Category"

select with a lower case


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

@brianteeman
Copy link
Contributor Author

unrelated to module position. will be addressed in a dedicated pr

@sandewt
Copy link
Contributor

sandewt commented Feb 24, 2023

I have tested this item ✅ successfully on 5448539


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

2 similar comments
@viocassel
Copy link
Contributor

I have tested this item ✅ successfully on 5448539


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

@alikon
Copy link
Contributor

alikon commented Feb 24, 2023

I have tested this item ✅ successfully on 5448539


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

@joomla-cms-bot joomla-cms-bot removed Language Change This is for Translators PR-4.2-dev labels Feb 24, 2023
@alikon
Copy link
Contributor

alikon commented Feb 24, 2023

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Feb 24, 2023
@richard67 richard67 added Language Change This is for Translators PR-4.2-dev labels Feb 24, 2023
@fancyFranci fancyFranci merged commit d7a276f into joomla:4.2-dev Feb 24, 2023
@fancyFranci fancyFranci added this to the Joomla! 4.2.9 milestone Feb 24, 2023
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Feb 24, 2023
@fancyFranci
Copy link
Contributor

Way better. Thanks!

@brianteeman
Copy link
Contributor Author

thx

@brianteeman brianteeman deleted the module_position branch February 24, 2023 23:25
Kostelano added a commit to JPathRu/localisation that referenced this pull request Mar 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Language Change This is for Translators

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants