Skip to content

[5.1] Add toolbar buttons in language installation toolbar#42610

Merged
Razzo1987 merged 5 commits intojoomla:5.1-devfrom
LadySolveig:language-installation-toolbar
Jan 16, 2024
Merged

[5.1] Add toolbar buttons in language installation toolbar#42610
Razzo1987 merged 5 commits intojoomla:5.1-devfrom
LadySolveig:language-installation-toolbar

Conversation

@LadySolveig
Copy link
Copy Markdown
Contributor

Pull Request for Issue #35268 .

Summary of Changes

Added toolbar buttons to the language installation view to go directly to the language manage view or the content language view - just like in the extension installation view.

Testing Instructions

Go to System => Install => Languages

Actual result BEFORE applying this Pull Request

No toolbar buttons for manage languages and content languages

grafik

Expected result AFTER applying this Pull Request

A new toolbar button to go directly to manage languages and to content languages

grafik

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

* add toolbar button link to manage languages
* add toolbar button link content languages
* remove set title for toolbar - is already handled in the parent::addToolbar
* remove set title for toolbar - is already handled in the parent::addToolbar
* remove duplicate access request - is already handled in parent::addToolbar
* add language strings for toolbar buttons
@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-5.1-dev labels Jan 4, 2024
Comment thread administrator/language/en-GB/com_installer.ini Outdated
* change order of language strings

Co-authored-by: Quy <quy@nomonkeybiz.com>
@brianteeman
Copy link
Copy Markdown
Contributor

Nothing against this PR but I don't see how it resolves #35268

@ghost
Copy link
Copy Markdown

ghost commented Jan 5, 2024

I have tested this item ✅ successfully on a76ae69

Thanks @LadySolveig


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

@torstenhoppe
Copy link
Copy Markdown

I have tested this item ✅ successfully on 46cd2fc


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

@CCMLanger
Copy link
Copy Markdown

I have tested this item ✅ successfully on 46cd2fc

works fine


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

@alikon
Copy link
Copy Markdown
Contributor

alikon commented Jan 8, 2024

Nothing against this PR but I don't see how it resolves #35268

a little step in the right direction

@alikon
Copy link
Copy Markdown
Contributor

alikon commented Jan 8, 2024

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jan 8, 2024
@Razzo1987 Razzo1987 merged commit 97ff305 into joomla:5.1-dev Jan 16, 2024
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jan 16, 2024
@Razzo1987
Copy link
Copy Markdown
Contributor

Thanks ;)

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

Labels

Feature Language Change This is for Translators

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants