Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added submenus to Extensions > Manage #8058

Closed
wants to merge 2 commits into from
Closed

Added submenus to Extensions > Manage #8058

wants to merge 2 commits into from

Conversation

pe7er
Copy link
Contributor

@pe7er pe7er commented Oct 11, 2015

This PR adds a submenus to the Extensions > Manage menu.

Most other menu items have submenus with direct links to all menu options.
e.g. Components > Banners, has 4 submenus.

Testing instructions

Before this PR

Extensions > Manage has currently no submenus

manage-before

After this PR

Extensions > Manage has submenus, which makes it possible to directly navigate to a submenu.
The order of the submenus should be the same as the order of the left menu items.
The links under the submenu items should be the same as the links in the left menu (they should point to the same pages).

manage-after

@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-staging labels Oct 11, 2015
@n9iels
Copy link
Contributor

n9iels commented Oct 11, 2015

I have tested this item ✅ successfully on bd0cbed


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

@zero-24
Copy link
Contributor

zero-24 commented Oct 11, 2015

@pe7er can you alpha sort the new language strings?

@joomla-cms-bot
Copy link

This PR has received new commits.

CC: @n9iels


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

@pe7er
Copy link
Contributor Author

pe7er commented Oct 11, 2015

Thanks for testing @n9iels !
Thanks for your recommendation @zero-24, I've alpha sorted the language strings.

@crommie
Copy link

crommie commented Oct 11, 2015

I have tested this item ✅ successfully on 98e13ac

Tested, works like it should. I like this one.


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

@zero-24
Copy link
Contributor

zero-24 commented Oct 11, 2015

I have tested this item ✅ successfully on 98e13ac

Works Thanks.


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

@zero-24
Copy link
Contributor

zero-24 commented Oct 11, 2015

RTC. Based on testing 😃


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Oct 11, 2015
@zero-24 zero-24 added this to the Joomla! 3.5.0 milestone Oct 11, 2015
@coolman01
Copy link

I have tested this item ✅ successfully on 98e13ac

Something I've missed for ever! Great stuff. I really hope this gets added in the enxt version!
Works perfectly.


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

@zero-24
Copy link
Contributor

zero-24 commented Oct 24, 2015

@coolman01 it is planed for 3.5 ;)

roland-d pushed a commit that referenced this pull request Oct 27, 2015
@roland-d
Copy link
Contributor

Thanks everybody, merged into 3.5-dev with commit 1ff1dfb

@roland-d roland-d closed this Oct 27, 2015
@pe7er pe7er deleted the extensions-manage branch November 5, 2015 00:08
@wilsonge wilsonge removed the RTC This Pull Request is Ready To Commit label Jan 17, 2016
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.

None yet

8 participants