Pie chart in metrics page truncates long email addresses #311

Closed
gitblit opened this Issue Aug 12, 2015 · 4 comments

Comments

Projects
None yet
2 participants
@gitblit
Owner

gitblit commented Aug 12, 2015

Originally reported on Google Code with ID 15

What steps will reproduce the problem?
1. let some committers with long email addresses commit code to a repository
2. browse the metrics page


What is the expected output? What do you see instead?
The committers name used as labels for pie slices do not fit into the graph. Emails
painted to the left of the pie get the user name truncated, emails painted on the right
side get the domain truncated. See attachment with pixel-lated email addresses.

What version of the product are you using? On what operating system?
0.5.2, but problem shows up also in 0.5.1

Please provide any additional information below.
Browsers if Firefox 5, o.s. Centos 5.5 x64, JVM Java HotSpot 1.6.0_24


Reported by gm.romanato on 2011-07-27 15:53:18


- _Attachment: chart.png
![chart.png](https://storage.googleapis.com/google-code-attachments/gitblit/issue-15/comment-0/chart.png)_
@gitblit

This comment has been minimized.

Show comment
Hide comment
@gitblit

gitblit Aug 12, 2015

Owner
Its easy enough to widen the fixed-width image.  Can you estimate how many pixels are
required?

Reported by James.Moger on 2011-07-29 12:19:31

  • Labels added: Milestone-0.6.0
Owner

gitblit commented Aug 12, 2015

Its easy enough to widen the fixed-width image.  Can you estimate how many pixels are
required?

Reported by James.Moger on 2011-07-29 12:19:31

  • Labels added: Milestone-0.6.0
@gitblit

This comment has been minimized.

Show comment
Hide comment
@gitblit

gitblit Aug 12, 2015

Owner
I would say at least 30%. 

How about making width default to a value 30% wider than current value and offering
both width and height as configurable parameters in gitblit.properties?

Reported by gm.romanato on 2011-07-29 13:18:37

Owner

gitblit commented Aug 12, 2015

I would say at least 30%. 

How about making width default to a value 30% wider than current value and offering
both width and height as configurable parameters in gitblit.properties?

Reported by gm.romanato on 2011-07-29 13:18:37

@gitblit

This comment has been minimized.

Show comment
Hide comment
@gitblit

gitblit Aug 12, 2015

Owner
Its currently 500.  I'll crank it up to 800 and change the layout of the page.

Reported by James.Moger on 2011-07-29 15:15:48

Owner

gitblit commented Aug 12, 2015

Its currently 500.  I'll crank it up to 800 and change the layout of the page.

Reported by James.Moger on 2011-07-29 15:15:48

@gitblit

This comment has been minimized.

Show comment
Hide comment
@gitblit

gitblit Aug 12, 2015

Owner
0.6.0 has been released.

Reported by James.Moger on 2011-09-28 02:09:21

  • Status changed: Fixed
Owner

gitblit commented Aug 12, 2015

0.6.0 has been released.

Reported by James.Moger on 2011-09-28 02:09:21

  • Status changed: Fixed

@gitblit gitblit closed this Aug 12, 2015

@fzs fzs modified the milestone: 0.6.0 Dec 13, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment