Skip to content

fix: Correct magenta spelling in the light colour map - #10

Merged
k-yle merged 1 commit into
k-yle:mainfrom
bkeepers:magenta-typo
Aug 12, 2026
Merged

fix: Correct magenta spelling in the light colour map#10
k-yle merged 1 commit into
k-yle:mainfrom
bkeepers:magenta-typo

Conversation

@bkeepers

@bkeepers bkeepers commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

COLOUR_MAP in data/planetiler/LightCharacteristics.java spells magenta as mangeta, and since encodeLx only appends a colour when the map contains the key, a light tagged seamark:light:colour=magenta silently loses its colour from the encoded description. One word, plus a test mirroring the existing single-colour case.

The TS library this file was ported from isn't affected; its colour table is keyed by abbreviation, so the tag-name map only exists in this port.

@bkeepers bkeepers changed the title Fix magenta spelling in the light colour map fix: Correct magenta spelling in the light colour map Aug 7, 2026

@k-yle k-yle left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

thanks :)

@k-yle
k-yle merged commit 53517b1 into k-yle:main Aug 12, 2026
3 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants