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

Legend is off-centre #515

Closed
aendra-rininsland opened this issue Aug 22, 2014 · 5 comments
Closed

Legend is off-centre #515

aendra-rininsland opened this issue Aug 22, 2014 · 5 comments
Labels
C-bug Category: This is a bug resolved maybe

Comments

@aendra-rininsland
Copy link
Member

I noticed this in relation to times/axisJS#9, but it seems the legend is slightly off-centre.

Using Photoshop, I've drawn where the centre of each element is, and they're clearly a bit unaligned:

offcentre

I may write a PR for this given that my project needs to have this fixed.

@masayuki0812
Copy link
Member

You're right and I think this has been fixed by this commit. Could you try the latest code? Thanks.

@masayuki0812 masayuki0812 added C-bug Category: This is a bug resolved maybe labels Sep 7, 2014
@aendra-rininsland
Copy link
Member Author

Looks pretty close, if not spot-on — via my not-very scientific use of Photoshop to find the centres of each element (Creating a selection to each edge, transforming its width to 1px, drawing vertical line. There's totally a better way to do this in SVG, I'm sure...), they still seem about a pixel off, though this could just be due to an issue converting vector to raster via screencap, or a rounding issue with the size of the graph I made:

screen shot 2014-09-07 at 14 47 56

I'm satisfied with it, nicely done! Closing issue — many thanks, will update axisJS now! :shipit: 👍

@aendra-rininsland
Copy link
Member Author

Actually, reopening because you have a testing framework and this shouldn't be a difficult thing to write a test for. Will try to do a pull request on Monday containing a test not involving silly stuff done in Photoshop!

@masayuki0812
Copy link
Member

I added a test for this issue. So, please let me close. Thanks.

@aendra-rininsland
Copy link
Member Author

Thanks for doing that! Have been swamped at work and totally forgot about writing that test!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: This is a bug resolved maybe
Projects
None yet
Development

No branches or pull requests

2 participants