? ? Pending

User tests: Successful: Unsuccessful:

avatar izharaazmi
izharaazmi
30 Mar 2017

Summary of Changes

  • Added an optional title parameter to addSeparator() method in backend menu class. Hence there are now two separator types:

    • Normal Separator (horizontal bar)
    • Labelled Separator
  • In the preset menu, converted the separators between menus to labelled separators. This helps identify which menu are site menus and which are administrator menus.

  • This labelled separators can also be used in the new custom backend menu by simply providing a label to the separator type menu item.

Testing Instructions

  • Install current staging and navigate to backend and see the drop down menu.which should look like

menu

  • Add a custom administrator menu and set it as active backend menu.
  • Add some items, separators with title and separators without title in custom backend menu
  • Match your findings with expected results below.
  • If there is no admin menu,the separator should also be hidden automatically.
  • Test this on RTL layout also.

Expected result

  • The separators with title should appear as labelled separators whereas the separator without title should be displayed as normal separator.
  • The separators should not be clickable

Documentation Changes Required

Backend menu manager documentation should be updated.

@infograf768 @brianteeman

avatar izharaazmi izharaazmi - open - 30 Mar 2017
avatar izharaazmi izharaazmi - change - 30 Mar 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 30 Mar 2017
Category Modules Administration Templates (admin)
avatar C-Lodder
C-Lodder - comment - 30 Mar 2017

Can you please make styling changes in the correct LESS file and compile it

avatar izharaazmi
izharaazmi - comment - 30 Mar 2017

Sorry, I've no idea how to do that. Please advise.

avatar brianteeman brianteeman - test_item - 30 Mar 2017 - Tested successfully
avatar brianteeman
brianteeman - comment - 30 Mar 2017

I have tested this item successfully on 67c3cb4


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

avatar C-Lodder
C-Lodder - comment - 30 Mar 2017

The LESS files are in the template directory. To compile, you're going to need to open up your terminal, cd to the Joomla root, and run php build/generatecss.php.

avatar brianteeman
brianteeman - comment - 30 Mar 2017

to expand on what @C-Lodder said

You should never edit any css file. Just edit the relevant less files and then run the generate command as @C-Lodder said and then this will generate the relevant css files as well

avatar izharaazmi
izharaazmi - comment - 30 Mar 2017

Thanks. I was trying to use PHPStorm LESS Watcher for no avail. It
displayed so many errors. Didn't know there was a script for that. Doing it
now.

avatar izharaazmi izharaazmi - change - 30 Mar 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 30 Mar 2017
Category Modules Administration Templates (admin) Modules Administration Templates (admin) Front End Templates (site)
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 30 Mar 2017

Appended from System Links: Menu Heading and Text Separator to Costum Menu, none of them is showen.

avatar C-Lodder
C-Lodder - comment - 30 Mar 2017

again, could you please remove the asterisk from your CSS.

*width: 100%;
^
*margin: -5px 0 5px;
^
avatar izharaazmi
izharaazmi - comment - 30 Mar 2017

@franz-wohlkoenig Separators at beginning or end or immediately after another separator - are never shown.

@C-Lodder Those are already there in the divider less definition, I thing for IE only rule. See administrator/templates/isis/less/bootstrap/mixins.less:502

I recommend we have a test on IE too.

avatar C-Lodder
C-Lodder - comment - 30 Mar 2017

@izharaazmi - we don't support IE7

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 30 Mar 2017

@izharaazmi please have a look:
1

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 30 Mar 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 30 Mar 2017

I have tested this item successfully on a4baa3a

System Links > Text Separator with Text or - in Title-Field works.


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

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 30 Mar 2017

Changed from Separator using - in Title to Name in Title no Name is shown. Looks like Cache but had cleared Browser-Cache:
bildschirmfoto 2017-03-30 um 17 58 05

avatar izharaazmi
izharaazmi - comment - 30 Mar 2017

@franz-wohlkoenig You need to use separator type instead of heading.

I tested changing title from - to text and also the other way around. All looks fine here.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 31 Mar 2017

@brianteeman can you please retest?

avatar joomlara joomlara - test_item - 31 Mar 2017 - Tested successfully
avatar joomlara
joomlara - comment - 31 Mar 2017

I have tested this item successfully on a4baa3a


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

avatar franz-wohlkoenig franz-wohlkoenig - change - 31 Mar 2017
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 31 Mar 2017

RTC after two successful testes.

avatar insitevision insitevision - test_item - 31 Mar 2017 - Tested successfully
avatar insitevision
insitevision - comment - 31 Mar 2017

I have tested this item successfully on a4baa3a


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

avatar rdeutz rdeutz - change - 2 Apr 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-04-02 19:16:42
Closed_By rdeutz
Labels Added: ?
avatar rdeutz rdeutz - close - 2 Apr 2017
avatar rdeutz rdeutz - merge - 2 Apr 2017
avatar ZoFx
ZoFx - comment - 22 May 2017

After updating to 3.7 I desperately tried to create unnamed separators in a custom backend menu. Only here on Github I found I could achieve this by naming it with a single hyphen (-). In my opinion, this is not at all intuitive. How are users supposed to know that?

What's more when named with a single hyphen, the menu item is quite hard to click in the menu item list. Of course I can select the item and use the "edit" button, but who the heck actually does it this way?

I think it would be far more elegant to have a "Use title as separator label" property. And this should be switched off by default, which would be more the default behaviour of a "separator" imho.

avatar brianteeman
brianteeman - comment - 22 May 2017

@ZoFx commenting on a closed issue will not be seen - please create a new issue

avatar ZoFx
ZoFx - comment - 22 May 2017

I will, thanks.

Add a Comment

Login with GitHub to post a comment