Tag, you're it!

A list of tags is logically just that, a list, making a list element is the correct semantic choice for a containing element. Span makes no sense in this context and appears to have been used simply to render the tags inline.

Using the wrong element to achieve a certain visual appearance is bad.

Get a free froghurt

This PR implements the correct html tag for the job at hand and throws list items inline using Bootstrap's purpose-made .inline class, saving on additional css.

The frogurt is also cursed

Bootstrap's padding on inline-list elements is slightly greater than one letterspace (the whitespace that exists naturally between span elements), so tags will be spaced slightly further apart after this change.

But you get a free topping

This is actually beneficial because natural whitespace can be affected in unpredictable ways by, for example, extensions that minify the html. That would remove natural whitespace causing tags to smash into each other, something which would be prevented by this change.

61c4547 30 Sep 2014

Make list of tags into a list ;)

nternetinspired - open - 30 Sep 2014

jissues-bot - change - 30 Sep 2014

Labels

Added: ?

Kubik-Rubik - test_item - 1 Oct 2014 - Tested successfully

Kubik-Rubik - comment - 1 Oct 2014

Tested successfully, good change!

How to get the list? Just add some tags to an article, activate the display of tags in the options of the component and open the article in the frontend. Their you will see an inline list of all tags associated with this article.

This comment was created with the J!Tracker Application at http://issues.joomla.org/.