?
Referenced as Pull Request for: # 5713
avatar brianteeman
brianteeman
13 Jan 2015

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.

avatar brianteeman brianteeman - open - 13 Jan 2015
avatar brianteeman brianteeman - change - 13 Jan 2015
Title
Question Code Style
Question Code Style - line break syntax
avatar zero-24
zero-24 - comment - 13 Jan 2015

@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.
avatar mbabker
mbabker - comment - 13 Jan 2015

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 />.

avatar rdeutz
rdeutz - comment - 13 Jan 2015

Protostar is xhtml so
would be correct,
is also allowed in HTML5. I would use
(with a space between r and /

avatar brianteeman
brianteeman - comment - 13 Jan 2015

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/

avatar brianteeman brianteeman - change - 13 Jan 2015
Status New Closed
Closed_Date 0000-00-00 00:00:00 2015-01-13 19:43:01
Closed_By brianteeman
avatar brianteeman brianteeman - close - 13 Jan 2015
avatar zero-24 zero-24 - close - 13 Jan 2015
avatar brianteeman
brianteeman - comment - 13 Jan 2015

See #5713


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5711.
avatar zero-24
zero-24 - comment - 13 Jan 2015

Is someone able to document it on the code style documentation?

done can you have a look into it?

joomla/coding-standards#94

avatar brianteeman
brianteeman - comment - 13 Jan 2015

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/

avatar zero-24 zero-24 - change - 7 Jul 2015
Labels Added: ?

Add a Comment

Login with GitHub to post a comment