RTC bug PR-5.2-dev Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
22 Aug 2024

Pull Request for Issue #38856 .

Summary of Changes

renderelement adds a trailing slash when adding css links

link is a void element and the trailing slash should not be used in html5 (it was only needed for xhtml

Testing Instructions

View the source of the webpage OR use https://validator.w3.org/
Note with chrome inspect is not the same as view source

Actual result BEFORE applying this Pull Request

all css links in the head of the page have a trailing slash

Expected result AFTER applying this Pull Request

no trailing slash

additional information
https://github.com/validator/validator/wiki/Markup-%C2%BB-Void-elements#trailing-slashes-in-void-element-start-tags-do-not-mark-the-start-tags-as-self-closing

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar brianteeman brianteeman - open - 22 Aug 2024
avatar brianteeman brianteeman - change - 22 Aug 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 22 Aug 2024
Category Libraries
avatar Quy Quy - test_item - 22 Aug 2024 - Tested successfully
avatar Quy
Quy - comment - 22 Aug 2024

I have tested this item ✅ successfully on 97f95b3


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43961.

avatar fgsw fgsw - test_item - 22 Aug 2024 - Tested successfully
avatar fgsw
fgsw - comment - 22 Aug 2024

I have tested this item ✅ successfully on 97f95b3


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43961.

avatar Quy Quy - change - 22 Aug 2024
Status Pending Ready to Commit
Labels Added: PR-5.2-dev
avatar Quy
Quy - comment - 22 Aug 2024

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43961.

avatar Quy Quy - change - 22 Aug 2024
Labels Added: RTC
avatar exlemor exlemor - test_item - 24 Aug 2024 - Tested successfully
avatar exlemor
exlemor - comment - 24 Aug 2024

I have tested this item ✅ successfully on b77c3ae

I have successfully tested this.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43961.

avatar softforge softforge - test_item - 24 Aug 2024 - Tested successfully
avatar softforge
softforge - comment - 24 Aug 2024

I have tested this item ✅ successfully on b77c3ae

Removes all the trailing cross-throughs in the


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43961.

avatar Quy
Quy - comment - 24 Aug 2024

Part of PR #43239

avatar brianteeman
brianteeman - comment - 24 Aug 2024

@Quy except that PR is wrong

avatar Quy Quy - change - 11 Sep 2024
Labels Added: bug
avatar Hackwar
Hackwar - comment - 12 Sep 2024

Thank you for your contribution @brianteeman!

avatar Hackwar Hackwar - close - 12 Sep 2024
avatar Hackwar Hackwar - merge - 12 Sep 2024
avatar Hackwar Hackwar - change - 12 Sep 2024
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2024-09-12 07:30:21
Closed_By Hackwar
avatar brianteeman
brianteeman - comment - 12 Sep 2024

Thanks

Add a Comment

Login with GitHub to post a comment