[4.0] Replace com_content 'Image Float' with 'Image Class#31017
[4.0] Replace com_content 'Image Float' with 'Image Class#31017drmenzelit merged 5 commits intojoomla:4.0-devfrom
Conversation
|
Looks a good proposal and I see that you have considered the legacy issues. Thanks |
Co-authored-by: Richard Fath <richard67@users.noreply.github.com>
|
With the inspection of HTML I see the field The image of the test is the logo Joomla! The Image Class field is perhaps more like a list of elements than text. |
|
I have tested this item ✅ successfully on 65f5d4b This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31017. |
|
The previous test is still valid because the commit after that has only changed order of the language string. I've added back the test result in the issue tracker. |
|
Please update the following and remove obsolete strings: |
|
Tested successfully, class is added. |
|
It seems the PR meanwhile has a conflict in file |
|
I've resolved the conflict. @ChrisHoefliger Could you mark your test result in the issue tracker here? https://issues.joomla.org/tracker/joomla-cms/31017 . Just use the "Test this" button, then select the appropriate test result and then submitt. Please do it in the same way with the other pull requests which you have tested today and where you haven't dome this yet. Thanks in advance. |
|
I have tested this item ✅ successfully on 1185c10 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31017. |
|
It still needs to alpha sort the language strings as mentioned by @infograf768 above. |
|
I have tested this item ✅ successfully on 1185c10 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31017. |
|
I've allowed myself to fix the language string ordering and I've restored the previous test results. |
|
Can be set to RTC as soon as automated tests have passed. |
|
I have tested this item ✅ successfully on 8374b20 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31017. |
|
@ciar4n Will this PR require documentation changes when being merged? If so, let me know and I set the corresponding label. If you could add information about that in the description in that case would be great, too. Thanks in advance for feedback. |
|
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31017. |
|
Thanks |
This is a rebase of #17402. Opened on request #30924 (comment)
Pull Request for Issue #17399 #30924 .
Summary of Changes
Replaces the 'Image Float ' field in com_contents with an 'Image Class' field
Testing Instructions
Navigate to the Images and Links tab in article edit. Set an image and add a utility class to the Image Class field . Check frontend and ensure class has been applied correctly.