[#25454] - [com_templates] Add prefix to class name in menu assignment
- Fixed in Code Base
- 20 Jul 2019
- Medium
- Build: staging
- # 25454
- Diff
- joomla:menu-assignment
- Pending continuous-integration/appveyor/branch Waiting for AppVeyor build to complete Details
- Success continuous-integration/drone/push Build is passing Details
- Success continuous-integration/travis-ci/push The Travis CI build passed Details
- Pending Hound Hound is busy sniffing around... Details
User tests: Successful: Unsuccessful:
Pull Request for Issue #24605.
Summary of Changes
Add prefix to class name to prevent conflicting with existing classes.
Testing Instructions
Create a new Menu named Hidden and set the menutype to "hidden".
Add a menu item to that menu.
Open a template style and check the Menu Assignment tab.
Expected result
A visible checkbox to associate the template style with the created menu item.
Actual result
No checkbox visible. Reason for that is the menu alias will be added to the Input tag as a class - without any prefix. So it becomes input[type=checkbox].hidden which is a class used by bootstrap to hide stuff.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_templates |
| Title |
|
||||||
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25454.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25454.
| Status | Pending | ⇒ | Ready to Commit |
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25454.
| Labels |
Added:
?
?
|
||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2019-07-20 19:00:10 |
| Closed_By | ⇒ | HLeithner |
thx
I have tested this item✅ successfully on 06acd66
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25454.