mapbox / mapbox/mapbox-gl-styles

Mapbox styles should omit ways rated SAC scale 5 and 6

Open
#338 1 comment 2 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
JavaScript
Stars
315
Forks
143
PR merge metrics
No merged PRs in 30d

Description

SAC scale 5 and 6 trails should be omitted from Mapbox Outdoors (not included out-of-the-box). Mapbox users that deploy Mapbox maps should have to make a conscious decision to include these trails in their maps (e.g. if they are making a specialized mountaineering map).

Trails that are tagged with a 5 or 6 SAC scale rating require special equipment and training, and novice hikers may get stuck on these trails.

Example

Current Style Differences are Insufficient

Mapbox styles these trails differently, but not in a way that would make it obvious to a novice or even experienced hiker the trail isn't passable. There may be no style strong enough to do so.

Trails Cannot be Edited Out of the Style

It is also not possible to remove these trails using a custom Mapbox style on mapbox.com. Even if it were possible, this seems insufficient - these trails should not appear on Mapbox styles intended for outdoor apps by default, since many developers will show them without understanding the danger.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

No file or test is named in the issue. Start by locating the Mapbox Outdoors style definition and its trail layer filters, then inspect how SAC scale tags are represented; done means SAC scale 5 and 6 trails are omitted by default while other trails remain available.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
frontend
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.