[#8311] - More unneeded mootools calls
- Closed
- 10 Nov 2015
- Medium
- Build: staging
- Easy Test
- # 8311
- Diff
- dgt41:useless.mootools2
- Success Human Test Results: 2 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
More unneeded mootools calls
Testing
On a fresh installation go to global config and make sure that Mouse-over Edit Icons for is set to Modules
Go to the left panel on the modules and select Site
With go to front end and try to edit a module. Try to save it. Everything should function as usual.
There are also 3 more calls for mootools on some modal pages which I don’t have an easy way for testing. BUT by checking the other scripts loaded there and the content of those pages it is obvious that mootools is not required. So request for review on those, unless someone can get some testing instructions...
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
No B/C break there. The pages don’t need mootools so they shouldn’t load it. Now if the parent needs to load it with mootool it should load it there otherwise the modal will not work. These are pages that load on a modal so their functionality is pretty defined by the scripts loaded and those script don;t require mootools
Great. If i got some time today i Can test it. Thanks
I have tested this item 
successfully on 92eb9b6
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8311.
I have tested this item successfully on 92eb9b6
Works good. Thanks
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8311.
| Milestone |
Added: |
||
| Status | Pending | ⇒ | Ready to Commit |
| Easy | No | ⇒ | Yes |
| Milestone |
Added: |
||
| Category | ⇒ | JavaScript |
| Labels |
Added:
?
|
||
wilsonge
- | Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-11-10 22:54:17 |
| Closed_By | ⇒ | wilsonge |
| Labels |
Removed:
?
|
||
Do we not Need it for B/C or i'm wrong?