-
Notifications
You must be signed in to change notification settings - Fork 177
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
Grid lines thick and a bit obtrusive with 1.8.0 #638
Comments
This is very strange. First, I can not reproduce it on my computer (also with Firefox). Second, in 1.8.0, out of the box we use exactly the same color, opacity and stroke width as in 1.7.0. |
Update: now, I can reproduce it. On my computer the problems starts with zoom level |
Interesting. So it just happens that with my settings it happens one zoom level higher than for most others. That explains why there were no issues on this. It does seem like an anti-aliasing issue. I am not sure if this is still needed, but here is a jsfiddle: https://jsfiddle.net/drgrice1/1fL45apq/4/ Thanks for looking into this. |
Ok, fixed in the sources (branch develop) |
I can confirm that the issue is fixed with the develop branch. Thanks. |
With the changes to the grid in version 1.8.0, the grid lines have become considerably thicker than before, and are rather obtrusive. Aside from the lack of visual appeal, the grid can obscure other elements, primarily text labels on the axis. I noticed that if I zoom in with the browser (Ctrl-+), the grid lines suddenly go back to looking thin like they did before with previous versions of jsxgraph. Am I missing a new setting that can fix this? I tried many combinations of the new themes and other settings, and nothing helped.
Here are some screenshots demonstrating the difference. These were taken with Firefox, but I see the same behavior in Google Chrome.
The first screenshot is with 1.8.0 at the default zoom level:
The next is with 1.8.0 at a zoom of 120%:
The next is with 1.7.0 at the default zoom level:
Finally, with 1.7.0 at 120% zoom:
Here is the minimal code used for these (of course the version changed for the 1.7.0 screenshots):
The text was updated successfully, but these errors were encountered: