[#16135] - Improve imagelist custom field
- Fixed in Code Base
- 22 May 2017
- Medium
- Build: staging
- # 16135
- Diff
- PhilETaylor:Issue16121
User tests: Successful: Unsuccessful:
Pull Request for Issue #16121 .
Summary of Changes
Tidy up and bugfix custom field imagelist
Testing Instructions
Create a new Custom Field List of Images for the Content. Create a new article and select an image for the field.
See the article in frontend and see the image path:
src="/Joomla_3.7.1/images**///**image-name.gif"
Apply patch
Expected result
src="/Joomla_3.7.1/images/image-name.gif"
NOTE
Coded without testing
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End Plugins |
| Labels |
Added:
?
|
||
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16135.
I have tested this item
Patch ok for me. Thanks.
Is it a good idea also add the ALT tag with the field Label text ?
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16135.
When there is no class, which is preferred with space or no space before /? "/> or " />
I think WITH the space is "more right" ?
Ref: https://www.tutorialspoint.com/xhtml/xhtml_syntax.htm
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16135.
| Status | Pending | ⇒ | Ready to Commit |
| Status | Ready to Commit | ⇒ | Pending |
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16135.
| Status | Pending | ⇒ | Ready to Commit |
RTC after two successful tests.
They've been all done in Joomla 4. I'm only asking on this so it doesn't need to be addressed when staging is merged into 4.0-dev
| Labels |
Added:
?
|
||
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16135.
rdeutz
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-05-22 18:20:31 |
| Closed_By | ⇒ | rdeutz |
I have tested this item✅ successfully on 252d94e
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16135.