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

Space between the dashes on the zero line is not transparent #3821

Closed
holetest opened this issue Jan 23, 2017 · 8 comments
Closed

Space between the dashes on the zero line is not transparent #3821

holetest opened this issue Jan 23, 2017 · 8 comments

Comments

@holetest
Copy link

If I use the borderDash property with gridLines, the space between the dashes on the zero line is not transparent. Please look at this image: https://i.stack.imgur.com/T82Zc.png

Here is a CodePen: http://codepen.io/anon/pen/bgqyjW

I reproduced this issue on Firefox and Chrome on my Windows PC only when I resize the setting "change the size of text,apps and other items" to 125% in display settings. I am using ChartJS version 2.4

@etimberg
Copy link
Member

Try

options: {
  scales: {
    xAxes: [{
      gridLines: {
        drawBorder: false
      }
    }]
  }
}

@holetest
Copy link
Author

Still there: http://imgur.com/KLBORMG.png

It's like if the gridLines are still generated above even if it's the zeroline.

@etimberg
Copy link
Member

in the end, what look are you trying to achieve with that first gridline?

@holetest
Copy link
Author

Nothing in particular, that's why I'm pretty sure this is a bug. The gridline should not be displayed on top of the zero line especially if there's a border: http://i.imgur.com/4lC8DXT.png

@hackape
Copy link

hackape commented Feb 7, 2017

Got same issue here, dashed gridLines overlap zeroLine (which is the border).

@hackape
Copy link

hackape commented Feb 7, 2017

@holetest but these is a work-around, you can set zeroLineColor: 'rgba(0, 0, 0, 0)'

@lndobryden
Copy link
Contributor

I have a PR that exposes border dash options for the zero line that should alleviate this issue.
#4019
I'm working on getting a codepen of plnkr together to showcase how this works.

@etimberg
Copy link
Member

I believe this was resolved with #4019 which will release in v2.6.0

@etimberg etimberg added this to the Version 2.6 milestone May 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants