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

Jump to bottom

Pie chart in metrics page truncates long email addresses #311

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

Pie chart in metrics page truncates long email addresses #311

gitblit opened this issue Aug 12, 2015 · 4 comments
Assignees
@gitblit
Labels
🐱 Defect Priority-Medium Status-Fixed
Milestone
0.6.0

Comments

@gitblit
Copy link
Collaborator

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
Copy link
Collaborator Author

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
Copy link
Collaborator Author

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 gitblit self-assigned this Aug 12, 2015
@gitblit
Copy link
Collaborator Author

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
Copy link
Collaborator Author

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 as completed Aug 12, 2015
@flaix flaix 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
Assignees

@gitblit gitblit

Labels
🐱 Defect Priority-Medium Status-Fixed
Projects
None yet
Milestone
0.6.0
Development

No branches or pull requests
2 participants
@gitblit @flaix