? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
29 Dec 2017

Sure I did this before but this PR removes the use of a closing slash on a br tag as this is not correct for html5

avatar brianteeman brianteeman - open - 29 Dec 2017
avatar brianteeman brianteeman - change - 29 Dec 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 29 Dec 2017
Category Administration Language & Strings Templates (admin) JavaScript Front End Templates (site)
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 30 Dec 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 30 Dec 2017

I have tested this item successfully on b0f30f9


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

avatar Anu1601CS
Anu1601CS - comment - 30 Dec 2017

If We are using xhtml systems that generate HTML based on XML generators. XHTML Elements Must Always Be Closed.
So, use of closing tag on br is more fine to use because it support both html5 and xhtml


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19228.
avatar brianteeman
brianteeman - comment - 30 Dec 2017

Joomla 4 is html5 not XHTML

avatar Anu1601CS
Anu1601CS - comment - 30 Dec 2017

Ok done!


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19228.
avatar Anu1601CS Anu1601CS - test_item - 30 Dec 2017 - Tested successfully
avatar Anu1601CS
Anu1601CS - comment - 30 Dec 2017

I have tested this item successfully on b0f30f9


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

avatar franz-wohlkoenig franz-wohlkoenig - change - 30 Dec 2017
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 30 Dec 2017

Ready to Commit after two successful tests.

avatar wilsonge wilsonge - change - 30 Dec 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-12-30 16:50:40
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 30 Dec 2017
avatar wilsonge wilsonge - merge - 30 Dec 2017
avatar brianteeman
brianteeman - comment - 30 Dec 2017

Thanks

Add a Comment

Login with GitHub to post a comment