Feeling Lucky
?
[#20001] - Add support for custom layout name to image manager JS
- Closed
- 24 Oct 2020
- Medium
- Build: staging
- # 20001
- Diff
- ggppdk:imageman_support_custom_layout_name
- Success hound No violations found. Woof! Details
- Success continuous-integration/drone/pr the build was successful Details
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
- Success continuous-integration/appveyor/pr AppVeyor build succeeded Details
User tests: Successful: Unsuccessful:
Summary of Changes
If you create a custom-named layout override for
com_media/views/imageslist/tmpl/default.php
e.g. create one with name mydefault
templates/JTEMPLATENAME/html/com_media/imageslist/mydefault.php
and then load the above via an override
templates/JTEMPLATENAME/html/com_media/images/default.php
then JS of image manager will force to return to default template every time you navigate to a folder or every time you click the "Up" folder button
Testing Instructions
Expected result
Actual result
Documentation Changes Required
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_media JavaScript |
| Labels |
Added:
?
|
||
brianteeman
- comment
- 24 Oct 2020
this should be closed
Quy
- | Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-10-24 18:05:06 |
| Closed_By | ⇒ | Quy |
I do not really need this PR, since i am overriding JS method
window.ImageManager.setFrameUrl()thus i am able to keep the correct layout, regardless of this PR
but for sake of being proper, Mediamanager JS should not be hardcoded to only using 'default' layout