Fixed Thread view tag display

[Steve F]

Affected version

2.0

Could use some separating when the tags wrap.
iPhone 7 Plus

 

[Kier]

Short of changing the margin on the tags, I'm not sure how to tackle this - but the margin is specifically set so that tags don't take up more or less space than regular text, leaving a bit of a conundrum.

 

[Bionic Rooster]

Short of changing the margin on the tags, I'm not sure how to tackle this

Maybe limit the number of tags per post, that example has nine of which several are redundant.

 

[Steve F]

Short of changing the margin on the tags, I'm not sure how to tackle this - but the margin is specifically set so that tags don't take up more or less space than regular text, leaving a bit of a conundrum.

Changing .tagItem padding and margin to:

.tagItem {
        padding: 0 6px 1px;
        margin: 0 0 1px;
}

Seems to work alright from what I see. The current top padding makes the tag text seem lower in it's element than it should be.

padding: 1px 6px;

padding; 0 6px 1px;

After that change there is 1px extra room to add back to margin. Which ends up looking like:


Above is changing maring: -1px 0; to margin: 0 0 1px; though 2px looks a tad better.


Just some feedback

 

[Chris D]

Thanks for the feedback and the suggestion. It seems reasonable so we'll go with it for the next release.