Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#15435] - [40] br not br slash in html5

? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
20 Apr 2017

The tag is just br in html5 there is no need for the closing slash and it is actually invalid

avatar brianteeman brianteeman - open - 20 Apr 2017
avatar brianteeman brianteeman - change - 20 Apr 2017
Status New ⇒ Pending
avatar joomla-cms-bot joomla-cms-bot - change - 20 Apr 2017
Category ⇒ Administration com_associations Front End com_content
avatar Twincarb Twincarb - test_item - 20 Apr 2017 - Tested successfully
avatar Twincarb
Twincarb - comment - 20 Apr 2017

I have tested this item ✅ successfully on 6e2087d

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

avatar RonakParmar
RonakParmar - comment - 21 Apr 2017

I have tested this item ✅ successfully on 6e2087d

Done code review

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

avatar RonakParmar RonakParmar - test_item - 21 Apr 2017 - Tested successfully
avatar franz-wohlkoenig franz-wohlkoenig - change - 21 Apr 2017
Status Pending ⇒ Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 21 Apr 2017

RTC after two successful tests.

avatar laoneo
laoneo - comment - 21 Apr 2017

Don't understand why it is invalid?

avatar C-Lodder
C-Lodder - comment - 21 Apr 2017

@laoneo it's not invalid. It's optional. I decided yo remove the backslash on all void element in prevois PR so it's best to stay consistant

avatar wilsonge wilsonge - change - 21 Apr 2017
Status Ready to Commit ⇒ Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 ⇒ 2017-04-21 20:08:38
Closed_By ⇒ wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 21 Apr 2017
avatar wilsonge wilsonge - merge - 21 Apr 2017
avatar brianteeman
brianteeman - comment - 21 Apr 2017

thanks

Add a Comment

Login with GitHub to post a comment