Skip to content

Adjusting park labels - #1302

Draft
wmisener wants to merge 4 commits into
osm-americana:mainfrom
wmisener:wmisener/parks_adjustments
Draft

Adjusting park labels#1302
wmisener wants to merge 4 commits into
osm-americana:mainfrom
wmisener:wmisener/parks_adjustments

Conversation

@wmisener

Copy link
Copy Markdown
Collaborator

It's now been discussed a couple of times on Slack that park labels are too prominent (see #1290), especially at low z levels. This is somewhat exacerbated by the pending addition of Native reservations (#1289), which adds a second label style at low z that makes parks appear too prominent by comparison. This PR explores ways to de-emphasize parks at low z, perhaps by decreasing their font weight:
Current:
Screenshot 2025-11-14 at 9 22 15 PM
This PR:
Screenshot 2025-11-14 at 9 33 11 PM

Care will be needed to make sure the parks stay sufficiently prominent at high z however, maybe by switching back to bold at some zoom level and/or just by increasing the text halo.
Current:
Screenshot 2025-11-14 at 9 46 55 PM
This PR:
Screenshot 2025-11-14 at 10 16 01 PM

This should probably be done in tandem with an attempt to display parks via their rank at low z, as suggested in #1290.

Comment thread src/layer/park.js Outdated
layout: {
"text-field": Label.localizedName,
"text-font": ["Americana-Bold"],
"text-font": ["Americana-Regular"],

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Currently we’re boldfacing labels of other fills without icons, such as ponds, at high zoom levels, whereas most POIs end up in normal weight because we only label them as points with icons. I think this is a useful treatment to keep. Can we vary the font by zoom level? Note that you’d need to use a literal expression inside each stop of an interpolate or step expression.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants