-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[4.0] Menu items modal: adding missing filters #30087
Conversation
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. |
The menu dropdown is at the top and not listed with the other dropdowns. |
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. |
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.
This is fine. It's designed this way in backend. com_menu has a specific searchtools layout. |
rtc This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/30087. |
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 |
See #30097 |
Thanks! |
…outs * '4.0-dev' of github.com:joomla/joomla-cms: (612 commits) [4.0] Smart Search: Fixing ordering, order direction and disabled button (joomla#29474) [4.0] Generate routed Modal links for iframes when not on the root (joomla#30007) [4.0] Get menu directly in com_tags menu route helper (joomla#30039) Remove collapse when resizing from mobile to desktop (joomla#30132) [4.0] Wrap component output in `main` element to make Cassiopeia more accessible (joomla#29870) [4.0] Webauthn gmp warning (joomla#29731) [4.0] Refactor to return early, remove if depths and throw NotAllowed (joomla#29694) [4.0] CLI help text (joomla#29811) Feature/draggable typo fixes (joomla#29987) [4.0] Removing unnecessary workaround in finder indexer (joomla#30037) [4.0] Optimizing Smart Search for larger content (joomla#30008) [4.0] Fix js ajax for pre update checker (joomla#29980) [4.0] Cassiopea: Fixing modals custom-select fields display (joomla#30097) [4.0][com_fields] Fix draggable sorting (joomla#30094) [4.0] Correct incorrect @return documentation (joomla#30092) [4.0] Menu items modal: adding missing filters (joomla#30087) short to long php open tags with echo (joomla#30089) Use new Toolbar (joomla#30085) [4.0] Center status/date created headers (joomla#29249) [4.0] Fix Cassiopea searchtools alignment in modals (joomla#30077) ... # Conflicts: # administrator/components/com_templates/src/View/Template/HtmlView.php # installation/sql/postgresql/base.sql # libraries/src/Application/AdministratorApplication.php # libraries/src/Application/SiteApplication.php
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
Frontend
Expected result AFTER applying this Pull Request
Backend
Frontend