Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Avoid linebreaks within the labels in some Tag Clouds (patch included) #905

Closed
kurakin opened this issue Aug 5, 2009 · 9 comments
Closed
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Milestone

Comments

@kurakin
Copy link

kurakin commented Aug 5, 2009

I think, we should avoid of line breaks within the labels in some (not all) Tag Clouds views, especially in tables "Length of Visit" and "Pages per Visit". In the same time, line breaks within the labels in other Tag Cloud tables, such as "Visitor Browsers", "List of Keywords" and so on, should be allowed.

The proposed patch implements this behavior, offering the new table view property dont_break_labels. See before.png and after.png snapshots.

Take attention: this patch also offers little cosmetic improvement -- corrected line align in Tag Cloud. Left indents of lines are equal now (see snapshots).

@kurakin
Copy link
Author

kurakin commented Aug 5, 2009

Attachment: Avoid linebreaks within the labels in Length of Visit and Pages per Visit tag clouds, plus a little cosmetic change
PiwikAvoidLinebreak.patch

@kurakin
Copy link
Author

kurakin commented Aug 5, 2009

Attachment:
before.png

@kurakin
Copy link
Author

kurakin commented Aug 5, 2009

Attachment:
after.png

@pebosi
Copy link
Contributor

pebosi commented Aug 13, 2009

i think the additional function "disallowLabelsBreak" could be unattended. just do it with css.

@kurakin
Copy link
Author

kurakin commented Aug 14, 2009

Replying to pebosi:

i think the additional function "disallowLabelsBreak" could be unattended. just do it with css.

I need to disallow linebreaks for two tables only: "Length of Visit" and "Pages per Visit". Linebreaks in all other tables should stay allowed. So I propose new property "dont_break_labels" to disable linebreaks selectively.

Thereto, disallowed linebreaks table can appears on the same page (dashboard) with allowed linebreaks table. These shouldn't affect each other. I don't no, how to solve it with pure css.

@robocoder
Copy link
Contributor

Would it be bad if we used the nowrap for all tag clouds?

@kurakin
Copy link
Author

kurakin commented Aug 16, 2009

Replying to vipsoft:

Would it be bad if we used the nowrap for all tag clouds?

It should be good enough for me at this time (I tried this about 3 mounths ago).

However, the original solution proposed in this ticket is just little more flexible. Who knows... may be in the future Piwik will have the tables with very long labels, requiring linebreaks anyway...

Though, at this time the simple solution -- nowrap for all tag clouds -- is satisfying for me too.

Thankee for your attention.

@robocoder
Copy link
Contributor

tackle in conjunction with #946

@robocoder
Copy link
Contributor

In [1567], fixes #905 - apply css changes

@kurakin kurakin added this to the Piwik 0.5 milestone Jul 8, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Projects
None yet
Development

No branches or pull requests

3 participants