[#19507] - [4.0] Split the rules field to 2 layouts
- Fixed in Code Base
- 4 Feb 2018
- Medium
- Build: 4.0-dev
- # 19507
- Diff
- Anu1601CS:check-layout
User tests: Successful: Unsuccessful:
Summary of Changes
Splits the HTML part (JLayout) into two files.
Reasons:
1: less code per file
2: more efficient caching (less allocate memory, etc)
Testing Instructions
1: Apply patch
2: Go to Admin->Global Configuration
3: Check Permissions working as before
Expected result
This is code improvement
Actual result
Same as before
Documentation Changes Required
Yes
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Layout Libraries |
| Labels |
Added:
?
|
||
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2018-02-04 23:49:06 |
| Closed_By | ⇒ | wilsonge |
Thanks!
Why was that Jlayout placed into layouts/joomla/form/field/rules/tabs.php and not like most other field layouts into layouts/joomla/form/field/rules.php.
Imho it looks like it was originally planned to have multiple layouts but in the end it was just one.
Thanks. I'll move it then in my PR
@Anu1601CS Please check my comments here and please make a PR to correct the doc blocks. Thanks
Removed ACL tooltips due to remove in base branch.