[4.0] Reorganise administrator menu items - extensions manager #10657

izharaazmi · 2016-05-28T15:37:09Z

Summary of Changes

Reorganised the menu items under the extension manager top level menu item. As seen in the past few release we are no longer supporting Joomla core update via extensions manager, that is a good move. Having a good explanation from @brianteeman and @mbabker I was convinced well that these two need to operate independently with no internal handovers.

However, I can understand the annoyance caused to Brian and Michael explaining this to everyone separately over and again. I suggest to make some visual change that calls for the users' action.

With this reorganisation, people would feel that there is some change right there. Investing little they would be able to see both options alongside and tend more towards using appropriate option rather than assuming "I did it before, why not now!".

Testing Instructions

New testers only: Please follow the testing instructions at Testing Joomla! patches
Every link should open the correct page.
Changing permissions on 'com_joomlaupdate', 'com_postinstall' and 'com_installer' should hide the respective menu items correctly.

~~PS: Asking for password on Joomla Update seems unnecessary and annoying as well. If we really want to be sure it could be just a confirm checkbox or a button.~~ Irrelevant here.

…e distinguishable at the first sight. Lets not confuse people showing thing same visually, but changing things internally.

izharaazmi · 2016-05-28T15:56:51Z

Old menu looked like:

With this PR they look like:

brianteeman · 2016-05-28T16:00:33Z

I much prefer that we don't continue with hard coding the admin menu and finally have a real admin menu manager that is configurable

izharaazmi · 2016-05-28T16:11:17Z

That would be better. J3.6 itself or leave it for 3.7!

brianteeman · 2016-05-28T16:15:17Z

Too late for 3.6 please see #5492

izharaazmi · 2016-05-28T16:23:01Z

Wow! That's my Sunday treat 👍 I'll surely read all that tomorrow and make up my mind around.

Well, in that case we may consider this small change for now. Right?

zafarnayab · 2016-06-15T07:40:05Z

I have tested this item ✅ successfully on a2bfff3

I've tested this .
It works Successfully...
All link works Perfectly same as it works before Patched.

While revoking Permission on 'com_joomlaupdate', 'com_postinstall' and 'com_installer' it hidden for the respective User.

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10657.}

joomla-cms-bot · 2016-06-19T07:25:03Z

This PR has received new commits.

CC: @zafarnayab

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10657.}

izharaazmi · 2016-06-19T07:41:56Z

Can we remove label "Language Change" as this no longer requires one!

brianteeman · 2016-06-19T10:28:00Z

Removed as requested (note it is still too late for 3.6)

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10657.}

izharaazmi · 2016-06-19T10:39:40Z

Well, if protocols says I must obey.

May I ask under which specific reason this is late, coz IMO, this is just a reorganisation of existing menu items only. No additions, no removal of anything at all and no b/c break, no new feature. Just visual change.

I assumed the language freeze was the issue, hence I modified. I must be missing something, please enlighten me.

About the backend menu manager (i had a look at #5492), it sounds great to me and that I'd gladly work on if suggested/assigned to. And that I think is too late for now and should be considered only for J3.7.

brianteeman · 2016-06-19T10:47:55Z

re #5492 I really want to see that happen so any help i can give you I will

re this PR my understanding that as we are in beta stage then we are
"feature complete" see
https://developer.joomla.org/news/586-joomla-development-strategy.html

Maybe @wilsonge can expand on that

On 19 June 2016 at 11:39, Izhar Aazmi notifications@github.com wrote:

Well, if protocols says I must obey.

May I ask under which specific reason this is late, coz IMO, this is just
a reorganisation of existing menu items only. No additions, no removal
of anything at all and no b/c break, no new feature. Just visual change.

I assumed the language freeze was the issue, hence I modified. I must me
missing something, please enlighten me.

About the backend menu manager (i had a look at #5492
#5492), it sounds great to
me and that I'd gladly work if suggested/assigned to. And that I think is
too late for now and should be considered only for J3.7.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#10657 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/ABPH8U0mxwppI0y72Zxi_HSchvwxVomJks5qNRxvgaJpZM4IpGX7
.

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

izharaazmi · 2016-06-19T10:57:11Z

IMO, this is not even a feature, however if you think so.
This could be really helpful to people messing with the new Joomla update method vs extension installer.

For #5492. I'll soon get into a strategy as well as some programming based on the discussion there plus my understanding. I'll get back to you as and when needed.

ghost · 2016-06-19T13:15:08Z

I have tested this item 🔴 unsuccessfully on 1b0008c

Firefox. Staging staging 2016-06-16.
Applied patch. Clicked on Manage. Clicked on System. First level closes. Second level remains open.

joomla-cms-bot · 2016-06-19T13:34:46Z

This PR has received new commits.

CC: @bertmert, @zafarnayab

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10657.}

ghost · 2016-06-19T15:09:39Z

I have tested this item 🔴 unsuccessfully on 967145d

Problems with mobile menu.

Resize window or use mobile device to see mobile menu.

Tip Extensions. That's OK because you have the sidebar menu in next view.

Tip System. That's not OK because this is a "dead end street".

izharaazmi · 2016-06-19T15:16:45Z

@bertmert This PR does not add anything different that is not already in
the cms.
Does it work as expected before this patch? Waiting for your images and
description.

Thanks.

ghost · 2016-06-19T15:18:48Z

See edits above.

brianteeman · 2016-06-19T21:48:24Z

I have tested this item 🔴 unsuccessfully on 967145d

I confirm the findings of @bertmert

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10657.}

NeerajTailored · 2016-07-12T11:09:34Z

I have tested this item ✅ successfully on 967145d

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10657.}

JamalTailored · 2016-07-12T11:12:41Z

I have tested this item ✅ successfully on 967145d

Tested using joomla staging branch

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10657.}

…avoid component level collision and dependency.

izharaazmi · 2017-04-11T15:44:46Z

Yes

infograf768 · 2017-04-11T15:46:09Z

A bit too late imho for 3.7.0. Has to be considered for 3.7.1 or 3.8

ghost · 2017-04-11T15:58:07Z

I have tested this item ✅ successfully on 740cfec

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/10657.}

Gavakshi · 2017-04-12T15:52:28Z

I have tested this item ✅ successfully on 740cfec

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/10657.}

jeckodevelopment · 2017-04-12T16:05:16Z

RTC

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/10657.}

brianteeman · 2017-04-12T16:07:05Z

This is a big change in the UI and should not be just set to RTC based on two tests that it works

mbabker · 2017-05-21T23:03:14Z

My first thought here is let's do this in 4.0 and leave potentially major workflow related changes alone in 3.x in favor of stability at this point (and moving menu items is a major workflow change).

brianteeman · 2017-05-22T07:55:19Z

I agree

tonypartridge · 2017-06-29T23:29:32Z

Agreed this should be moved to V4. Users won't be pleased with this change in 3.x since it's quite a workflow change.

But I think it's a great idea.

mbabker · 2017-07-01T15:18:44Z

Moved to 4.0 milestone for reasons stated above.

ghost · 2017-10-27T06:02:00Z

please reassign Milestone 4.0, thanks.

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/10657.}

ghost · 2017-10-30T08:03:56Z

@izharaazmi can you please resolve conflicting Files?

brianteeman · 2017-10-30T08:08:35Z

I would wait for the results of the UX team research into the menus

roland-d · 2018-07-22T19:40:02Z

Are we ready to move on this?

brianteeman · 2018-07-22T19:42:53Z

Well the template team still havent completed the menu and have arbitrarily reassigned the menus etc so I guess not

wilsonge · 2018-08-01T13:02:51Z

Now we have the configurable admin menu and a different structure in J4 I'm going to close this for now (it totally conflicts anyhow with the xml structure for admin menus - so it would need to be completely redone to test anyhow). We can create a new version if people still feel this is a good idea later on.

Reorganise menu items to make Joomla core update and extensions updat…

34aae48

…e distinguishable at the first sight. Lets not confuse people showing thing same visually, but changing things internally.

joomla-cms-bot added Language Change This is for Translators PR-staging labels May 28, 2016

izharaazmi mentioned this pull request Jun 14, 2016

Preventing user changing own group #10776

Merged

Fix menu nesting issue

a2bfff3

wilsonge added the Feature label Jun 18, 2016

Changes in favour of the J3.6 beta release and language freeze status.

1b0008c

brianteeman removed the Language Change This is for Translators label Jun 19, 2016

Make first level menu clickable

967145d

Changes to allow easy mobile view navigation. Reorganised further to …

07ba501

…avoid component level collision and dependency.

joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Apr 12, 2017

rdeutz added this to the Joomla 3.8.0 milestone Apr 12, 2017

mbabker modified the milestones: Joomla 4.0, Joomla 3.8.0 Jul 1, 2017

joomla-cms-bot changed the title ~~Reorganise administrator menu items - extensions manager~~ [4.0] Reorganise administrator menu items - extensions manager Oct 27, 2017

joomla-cms-bot removed this from the Joomla 4.0 milestone Oct 27, 2017

infograf768 added this to the Joomla 4.0 milestone Oct 27, 2017

joomla-cms-bot removed this from the Joomla 4.0 milestone Jan 17, 2018

brianteeman added this to the Joomla 4.0 milestone Mar 17, 2018

wilsonge closed this Aug 1, 2018

joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Aug 1, 2018

[4.0] Reorganise administrator menu items - extensions manager #10657

[4.0] Reorganise administrator menu items - extensions manager #10657

Conversation

izharaazmi commented May 28, 2016 • edited by joomla-cms-bot Loading

Summary of Changes

Testing Instructions

izharaazmi commented May 28, 2016

brianteeman commented May 28, 2016 via email

izharaazmi commented May 28, 2016 • edited Loading

brianteeman commented May 28, 2016 via email

izharaazmi commented May 28, 2016 • edited Loading

zafarnayab commented Jun 15, 2016

joomla-cms-bot commented Jun 19, 2016

izharaazmi commented Jun 19, 2016

brianteeman commented Jun 19, 2016

izharaazmi commented Jun 19, 2016 • edited Loading

brianteeman commented Jun 19, 2016

izharaazmi commented Jun 19, 2016

ghost commented Jun 19, 2016 • edited by ghost Loading

joomla-cms-bot commented Jun 19, 2016

ghost commented Jun 19, 2016 • edited by ghost Loading

izharaazmi commented Jun 19, 2016 • edited Loading

ghost commented Jun 19, 2016

brianteeman commented Jun 19, 2016

NeerajTailored commented Jul 12, 2016

JamalTailored commented Jul 12, 2016

izharaazmi commented Apr 11, 2017 via email

infograf768 commented Apr 11, 2017

ghost commented Apr 11, 2017

Gavakshi commented Apr 12, 2017

jeckodevelopment commented Apr 12, 2017

brianteeman commented Apr 12, 2017

mbabker commented May 21, 2017

brianteeman commented May 22, 2017

tonypartridge commented Jun 29, 2017

mbabker commented Jul 1, 2017

ghost commented Oct 27, 2017

ghost commented Oct 30, 2017

brianteeman commented Oct 30, 2017

roland-d commented Jul 22, 2018

brianteeman commented Jul 22, 2018

wilsonge commented Aug 1, 2018 • edited Loading

izharaazmi commented May 28, 2016 •

edited by joomla-cms-bot

Loading

izharaazmi commented May 28, 2016 •

edited

Loading

izharaazmi commented May 28, 2016 •

edited

Loading

izharaazmi commented Jun 19, 2016 •

edited

Loading

ghost commented Jun 19, 2016 •

edited by ghost

Loading

ghost commented Jun 19, 2016 •

edited by ghost

Loading

izharaazmi commented Jun 19, 2016 •

edited

Loading

wilsonge commented Aug 1, 2018 •

edited

Loading