Closed (fixed)
Project:
Active Tags
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
3 Feb 2010 at 14:56 UTC
Updated:
8 Mar 2010 at 13:10 UTC
Jump to comment: Most recent file
Comments
Comment #1
portulacaI can confirm the patch above helps with Popular tags.
But to stop the problem from appearing you need to do the same for "tag-tags" too. I'm attaching the patch.
btw why isn't this part themed through Drupal? We can't override the output easily, we have to hack the js file.
Comment #2
dragonwize commentedI don't really understand what the issue here is. Can someone please explain more or post a screen shot of the issue that you are trying to solve?
@portulaca: It is ran through the Drupal theme system so that you do not have to hijack or modify any of the modules files. See http://drupal.org/node/171213
Comment #3
portulacaI didn't know about that! Thanx for the clarification, it's very useful.
The problem is probably about display:inline that is applied to the tags, but there is no space between them so they all appear to be one long word so it doesn't wrap. Screenshot attached, the tags widen their container so it leaks to the texture area instead of wrapping to the next line.
I still think this should be patched since you'll always encounter this problem, if you set it to display:inline as default it makes sense to have the space between tags so they wrap if they reach the end of the container.
Users can float them to avoid this problem but that layout introduces more problems, I favor the inline approach.
Comment #4
dragonwize commentedThank you. Committed.