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

Thicker border in 3.5.0+ #9674

Closed
muss0v opened this issue Sep 24, 2021 · 2 comments · Fixed by #9678
Closed

Thicker border in 3.5.0+ #9674

muss0v opened this issue Sep 24, 2021 · 2 comments · Fixed by #9678

Comments

@muss0v
Copy link

muss0v commented Sep 24, 2021

Expected Behavior

Same as version 3.4.1.

Current Behavior

Since version 3.5.0 the border thickness has increased for the chart (but not legend).

Possible Solution

Decreasing the "borderWidth", but that also affects the border used in the legend which I don't want to change.

Steps to Reproduce

You can check this behaviour in the documentation (compare latest version vs 3.4.1.):
https://www.chartjs.org/docs/latest/samples/bar/border-radius.html (thicker border)
vs
https://www.chartjs.org/docs/3.4.1/samples/bar/border-radius.html (OK border)

https://www.chartjs.org/docs/latest/samples/bar/horizontal.html (thicker border)
vs
https://www.chartjs.org/docs/3.4.1/samples/bar/horizontal.html (OK border)

Might have to resize your screen for a better comparison (in the latest version the chart is much larger in the documentation).

Context

Since version 3.5.0 (also in 3.5.1) the border thickness has changed. It's more bolder now.

I always used "borderWidth: 2" which was perfect for my case, but now it's too thick. When I use "borderWidth: 1" it's thinner than before and also the legend border changes (which I don't want).

@etimberg
Copy link
Member

The only border related change I saw in 3.5.0 was #9452 which does not appear related.

@kurkle any ideas? Could this be a pixel alignment issue?

@kurkle
Copy link
Member

kurkle commented Sep 24, 2021

I suspect this comes from #9399

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

Successfully merging a pull request may close this issue.

3 participants