addition of a clear div for modal-footer to fix display issue reported in #9888 and closed #9897
refactory of the com_templates hardcoded bootsrap modals in default.php, to use JLayouts/modal (the issue was not using the Jlayout modal which includes core hack script to fix BS modal issues)
addition of each modal body and footer in a separated template
code style and better handler to detect when auto scrolling to be added and adjust modal height on small devices
Testing Instructions
TEST on current STAGING ONLY!
See #9888 (comment) for screenshots showing issue, and to help testing this patch.
In addition, test that all other modals (batch, users, versions...) are still working as before.
Same testing requirement as for PR #9817 (comment)
Note on com_templates admin
In component templates, admin side, modal buttons to be tested:
Copy template (collapseModal)
Rename File (renameModal)
Delete File (deleteModal)
New File (fileModal)
Manage Folders (folderModal)
Resize Image (resizeModal)
A code review by PLT is welcome! (as addition of 12 news view files...)
- Test like above described on com_template, Protostar (Hathor looks like don't use Modals).
Test batch on users, Note Categories, Menu Items, Articles, -Categories, Banners, -Categories, Contacts, -Categories, News Feeds, -Categories, Tags, Modules.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9908.
I have tested this item successfully on b041dfb
- Test like above described on com_template, Protostar (Hathor looks like don't use Modals).
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9908.