docs(visuals): rebuild the page around the map's Legend card - #18
Merged
Merged
Conversation
The legend now links to https://wiki.meshmapper.net/visuals/, so this page is the documented answer for every mark on the map. It was stale, and inverted on two points: pink was given as Active when pink is New, and orange as New when orange is TX coverage. Rewritten in the legend's own order (coverage squares, repeaters, links, signal) so the two can be read side by side. New sections for the repeater chips and their five states, the observer eye, group badges, the four neighbour link kinds and the Backbone overlay. Grid System, Ping Types and Private Repeaters are carried over verbatim. Colours are named with their Default-palette hex, under one note that the four colour vision palettes remap all of them. layers.md gains a Backbone row, the two missing neighbour link kinds and chip wording; duplicaterepeaterid.md carried the same wrong colours and is fixed to match. Verified against MeshMapper_Server dev/index.php at 91c3a2f. Where the handoff brief and the source disagreed, the source won; the differences are listed in the PR. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014HA5hZ2YdkCkgrNkxBrSRN
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.
The region map's legend now ends with a "What do these mean?" link to
https://wiki.meshmapper.net/visuals/(dev/index.php:20150, Serverdev@91c3a2f). That makesdocs/visuals.mdthe documented answer for every mark on the map, and it was stale, and on two points inverted: pink was given as Active when pink is New, and orange as New when orange is TX coverage.What changed
docs/visuals.mdis rewritten in the legend's own order (coverage squares, repeaters, links, signal) so the card and the page can be held side by side. The H1 now matches the nav entry.## Repeater Icons, which described marks that no longer exist.docs/layers.md: a Backbone row, the two missing neighbour link kinds (Reported, Uploaded), and chip wording on the Repeaters row.docs/duplicaterepeaterid.md: carried the same wrong colours (New | Orange Icon) and is corrected to match.Where the handoff brief and the source disagreed
Per the brief's own rule, the source won. These are worth a read before merging.
The dash/solid instructions were inverted
lineDashis assigned at:19725,:19985,:19995,:20004and never read. No polyline indrawLine(:20031-20034) or in the neighbour draws (:15804,:15832,:15854) passesdashArray, and the MapLibre shim only routes to its dashed layer onfilter: ['!=', ['get','dash'], ''](:1385,:1600). So every popup line and every neighbour link renders solid.The legend card paints Inferred, fresh (
:20127) and Route path (:20131) as2px dashedswatches, but both draw solid on the map. The legend and the renderer disagree. The wiki cannot fix this; this PR documents what the map actually draws. Worth a Server-side issue to decide which side moves.Other corrections
:4860); deuteranopia is aliased to protanopia (:14712), so three distinct blocks back them.:9291).:9485). Page avoids implying a majority.zoom >= 14ORpoint_count <= 18(:9896). Zoom only when both fail, with a fallback to fanning, so a group is never unopenable.:16641isif (repState === 'active') repState = 'backbone';so New keeps pink too.:16636,BB_LABEL_PCT:10050).:20077). The page says the shares add up to about half and never mentions a headline.:15405); the 7 isNB_RETENTION.inferred, admin-settable (:485). Written as settable.signal.mediumis#856404, a dark mustard (:14657). There is also a grey#6c757dfallback the brief omitted (:20025).:10069). Used as scale, not as region names.stale_repeater_hoursdefault 24Confirmed accurate in the brief: all five link hexes, all five chip hexes, the
BB_RAMPvalues and order, the 50% cut, and the group-as-one-pool rule.Verification
mkdocs build --strictclean, with the same deps CI installs. No new warnings against the baseline build ofmain(the two pre-existing INFO notes aboutregion_boundaries.mdandonboarding.mdare unchanged).layers.mdresolve to real anchors (#repeaters,#links-between-repeaters,#the-backbone), as dolayers.md#accessibilityand every in-page anchor.grepconfirms no page still says orange means New or pink means Active.🤖 Generated with Claude Code
https://claude.ai/code/session_014HA5hZ2YdkCkgrNkxBrSRN