I checked the coding style guide http://joomla.github.io/coding-standards/?coding-standards/chapters/html.md and I couldn't see an answer.
Which of the following is correct for joomla code style so we are consistent
<br/> <br> <br />
Current Staging contains
<br>
9 (in vendor libraries)
<br/>
26 in 10 files
<br />
111 in 51 files
Happy to do the PR to make it consistent just need to know what it should be.
Title |
|
I really don't remember if we were aiming to be XHTML compliant or not, but that should weigh in on things. As far as personal preference goes, I lean towards <br />
.
Protostar is xhtml so
would be correct,
is also allowed in HTML5. I would use
(with a space between r and /
Ok I will prepare a PR for <br />
Is someone able to document it on the code style documentation?
On 13 January 2015 at 19:30, Robert Deutz notifications@github.com wrote:
Protostar is xhtml so
would be correct,
is also allowed in HTML5. I would use
(with a space between r and /—
Reply to this email directly or view it on GitHub
#5711 (comment).
Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/
Status | New | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-01-13 19:43:01 |
Closed_By | ⇒ | brianteeman |
See #5713
Is someone able to document it on the code style documentation?
done can you have a look into it?
Thanks @zero-24
On 13 January 2015 at 21:10, zero-24 notifications@github.com wrote:
Is someone able to document it on the code style documentation?
done can you have a look into it?
joomla/coding-standards#94
joomla/coding-standards#94—
Reply to this email directly or view it on GitHub
#5711 (comment).
Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/
Labels |
Added:
?
|
@brianteeman
I think
<br>
is a good solution for html but it don't work for xhtml (<- but I'm not sure if we use any xhtml.)Here are some resorces:
http://stackoverflow.com/questions/1946426/html-5-is-it-br-br-or-br
It is also supported for all our supported browsers:
http://www.w3schools.com/tags/tag_br.asp
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5711.