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

Grid lines - add option to set thickness #733

Open
havatv opened this issue Jul 29, 2016 · 8 comments

Comments

@havatv
Copy link

commented Jul 29, 2016

When printing using 0.6.3 (on Ubuntu and Windows), the width of grid lines seems to scale according to map scale. On screen, the width of the grid lines do not scale when zooming.
When printing base maps, grid lines are useful for referencing, but they should be thin in order not to interfere with the base map.
It would be useful to be able to specify the printed width of the grid lines. Preferably a scale-independent width.
This seems to be related to #227, but the fix for that issue does not seen to solve this problem.

@dg0yt

This comment has been minimized.

Copy link
Member

commented Dec 2, 2018

Grid lines are printed with 0.1 mm at map scale which is thin. But they scale with printing scale. Right?

@havatv

This comment has been minimized.

Copy link
Author

commented Dec 2, 2018

Yes. When mapping according to ISOM with larger scales than normal (e.g. 1:5000 and larger) and base maps with a lot of detail, the the grid lines get too thick and tend to hide things. One option is to keep the 0.1 mm width but not scale the width of the grid lines when changing map scale for printing, another option is to make the thickness configurable.

@dg0yt dg0yt self-assigned this Dec 2, 2018
@dg0yt dg0yt added this to the v0.8.4 milestone Dec 2, 2018
@dg0yt dg0yt removed this from the v0.8.4 milestone Dec 10, 2018
@dg0yt dg0yt removed their assignment Dec 10, 2018
@dg0yt

This comment has been minimized.

Copy link
Member

commented Dec 10, 2018

A major sub-issue is fixed, but I'm not planning to work on custom grid line width soon.

@havatv

This comment has been minimized.

Copy link
Author

commented Dec 10, 2018

Avoiding scaling is a real improvement - thanks!
Being able to specify the width of the grid lines is not that important, so it is OK for me to close this issue now.

@dg0yt

This comment has been minimized.

Copy link
Member

commented Dec 11, 2018

I don't want to close it. It is a reasonable feature request. (It is even not very difficult to implement, but of course it touches GUI as well as the file format.) I just had to clarify that this feature won't be part of v0.8.4.

@dg0yt dg0yt closed this in 81f6da0 Dec 12, 2018
@havatv

This comment has been minimized.

Copy link
Author

commented Dec 12, 2018

@dg0yt, was it closed accidentally (closing is not consistent with #733 (comment) that you wrote two days ago)?

@dg0yt

This comment has been minimized.

Copy link
Member

commented Dec 12, 2018

Thanks for the hint. Commit 81f6da0 states that it fixes this issue and triggered an automatism. However, this commit statement was somewhat inaccurate.

@dg0yt dg0yt reopened this Dec 12, 2018
@dg0yt

This comment has been minimized.

Copy link
Member

commented Dec 13, 2018

The master and unstable builds now contain the line scaling fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.