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

Fixed last tick overlap for time scale #2749

Closed
wants to merge 1 commit into from
Closed

Fixed last tick overlap for time scale #2749

wants to merge 1 commit into from

Conversation

ankane
Copy link
Contributor

@ankane ankane commented Jun 10, 2016

First off, thanks for all the great work on this library!

This is a fix for #2599. It hides the last label unless it's equally spaced.

@panzarino
Copy link
Contributor

It seems that this PR has been open for a while, what do you think about it @etimberg @tannerlinsley @simonbrunel? To me this just seems like a less complete version of #2600

@etimberg
Copy link
Member

Agreed

@panzarino
Copy link
Contributor

Closing as #2600 seems to be a better implementation of the same solution

@panzarino panzarino closed this Jul 19, 2016
@ankane
Copy link
Contributor Author

ankane commented Jul 19, 2016

No worries, hopefully #2600 can be resolved soon.

I'm sure everyone is busy, but it'd be great if someone could take a look at #2748 as well. It fixes a bug labeled P0.

@ankane
Copy link
Contributor Author

ankane commented Aug 25, 2016

Any chance you'll revisit this? There hasn't been progress on #2600 for 2 months, and there are decent number of open issues around it. #3069 #2774 #2249

@markterrill
Copy link

markterrill commented Jan 5, 2017

Hi, I'd be happy to take anything that will fix it! #2600 seems dead.

Edit: Thanks @ankane your one liner hack (} else if (me.scaleSizeInUnits === 0) {) did the trick!

@andyb3
Copy link

andyb3 commented Feb 19, 2017

Thanks for this fix @ankane, this issue still hasn't been resolved and your solution seems to work very nicely!

@KonnikPahoni
Copy link

Hey everyone,

Still experiencing the issue.

@benmccann
Copy link
Contributor

No ticks should overlap each other. We're currently focused on 3.0, which has numerous bug fixes. If you can reproduce with 3.0.0-alpha please file a new bug with an interactive example to reproduce and we'll take a look

@KonnikPahoni
Copy link

Should they overlap in 2.9.3? This is the version I'm using.

@benmccann
Copy link
Contributor

Overlapping labels always seems like a bad experience to me. So they "shouldn't", but that being said I personally don't have time to make fixes to 2.x anymore. I know that 3.x does have some fixes such as #6927 that have the potential to fix this issue though I'm not sure if it does or not without testing

@KonnikPahoni
Copy link

In my case they overlap when I set min/max. I simply removed boundary ticks. I am a beginner so I can hardly contribute to solving this. Hopefully this will be resolved in 3.x!

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

Successfully merging this pull request may close these issues.

7 participants