? ? Pending

User tests: Successful: Unsuccessful:

avatar infograf768
infograf768
13 Jul 2020

Summary of Changes

Menus modals do not display filtering by Menu Type in backend and also in frontend.
Also, in frontend, no filtering by Parent Menu item

This PR corrects this so that filtering in modals is similar to filtering in menu tems Manager

Testing Instructions

Edit an article in backend and also in frontend.
Through CMS Content display a Menu Modal

Actual result BEFORE applying this Pull Request

Backend

Screen Shot 2020-07-13 at 10 31 44

Frontend

Screen Shot 2020-07-13 at 10 32 41

Expected result AFTER applying this Pull Request

Backend

Screen Shot 2020-07-13 at 10 21 15

Frontend

Screen Shot 2020-07-13 at 10 19 37

avatar infograf768 infograf768 - open - 13 Jul 2020
avatar infograf768 infograf768 - change - 13 Jul 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 13 Jul 2020
Category Administration com_menus Front End
avatar ChristineWk ChristineWk - test_item - 13 Jul 2020 - Tested successfully
avatar ChristineWk
ChristineWk - comment - 13 Jul 2020

I have tested this item successfully on d52f9ec


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

avatar Quy
Quy - comment - 13 Jul 2020

The menu dropdown is at the top and not listed with the other dropdowns.

avatar ceford ceford - test_item - 14 Jul 2020 - Tested successfully
avatar ceford
ceford - comment - 14 Jul 2020

I have tested this item successfully on d52f9ec

The front end looks strange with the Menu Select drop-down being full page width.


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

avatar infograf768
infograf768 - comment - 14 Jul 2020

The front end looks strange with the Menu Select drop-down being full page width.

Yep. Only way would be to create a specific css. Not sure what precisely as using flex is not my specialty. This would be for another pr.

The menu dropdown is at the top and not listed with the other dropdowns.

This is fine. It's designed this way in backend. com_menu has a specific searchtools layout.

Screen Shot 2020-07-14 at 07 34 21

avatar infograf768 infograf768 - change - 14 Jul 2020
Status Pending Ready to Commit
avatar infograf768
infograf768 - comment - 14 Jul 2020

rtc


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

avatar richard67 richard67 - change - 14 Jul 2020
Labels Added: ? ?
avatar infograf768
infograf768 - comment - 14 Jul 2020

Please merge as I have a PR for the css solving both the menutype custom-select display and the bcakground of the custom-select arrow

avatar infograf768
infograf768 - comment - 14 Jul 2020

See #30097

avatar richard67 richard67 - close - 14 Jul 2020
avatar richard67 richard67 - merge - 14 Jul 2020
avatar richard67 richard67 - change - 14 Jul 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-07-14 10:16:13
Closed_By richard67
Labels
avatar richard67
richard67 - comment - 14 Jul 2020

Thanks!

Add a Comment

Login with GitHub to post a comment