Repository navigation
Conversation
Finland tags its national road network highway=trunk, not motorway, so at the global trunk zoom of 6 it is absent when the whole country is in view. Only the geometry zoom moves.
pekkine
force-pushed
the
fi-trunk-road-zooms
branch
from
October 5, 2026 16:09
55a53d9 to
6ee4eae
Compare
|
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.




Implements the per-country minzoom from #221, for Finland. Adds a
pm:country=FIrule tohighwayZoomsIndexnext to the US ones, givinghighway=trunka minzoom of 3.Finland tags its national road network
highway=trunk, not motorway, so z3 to z5 carry almost none of it today. All of Finland fits on a phone screen at about z3, and that is the view where the road network is what lets a reader locate anything.Refs #221
Notes
Only the geometry zoom moves.
trunk_linkstays at 6,kindstaysmajor_road, and nothing changes in the default styles.@nvkelso raised in #221 that differing thresholds make roads start and stop at country boundaries. That happens here: the Finnish network ends at the Swedish and Norwegian borders, since those countries have no rule. The same rule can be added for them if that is wanted.
Short stretches right at the border also stay at the old zoom, since country assignment is per way and a way that is not fully inside the country polygon gets none. That is a small loss, since the border itself is drawn at these zooms and already gives the reader something to locate against.
Also adds a Finland bounding box to the
LayerTestcountry coder fixture, without which nopm:country=FIrule can be tested.AI Assistance
This PR was created with the help of Claude Code (Claude Opus 5). All code changes have been reviewed and understood by a human contributor.