Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

style(vector): update aerialhybrid style BM-1085 #987

Merged
merged 3 commits into from
Oct 7, 2024

Conversation

tawera-manaena
Copy link
Contributor

Motivation

As a Basemaps user, I want place names to be visually distinct based on their type. I want to be able to easily identify different geographic features based on their labels.

Modifications

  1. Deleted the following layers from the aerialhybrid.json stylesheet:

    Layers
    Place-Names
    Place-Names-3-12
  2. Copied the following layers from the topographic.json stylesheet into the aerialhybrid.json stylesheet:

    Layers
    Place-Names-13-Water
    Place-Names-13-Natural
    Place-Names-13-Place
    Place-Names-3-12-Water
    Place-Names-3-12-Natural
    Place-Names-3-12-Place
  3. Adjusted the styling of the new layers in the aerialhybrid.json stylesheet:

    🔵 Water

    Layer text-font text-color text-halo-color
    Place-Names-13-Water Open Sans Italic rgba(229, 248, 255, 1) rgba(16, 51, 64, 1)
    Place-Names-3-12-Water Open Sans Italic rgba(229, 248, 255, 1) rgba(16, 51, 64, 1)

    🟢 Natural

    Layer text-font text-color text-halo-color
    Place-Names-13-Natural Open Sans Italic rgba(238, 255, 229, 1) rgba(32, 64, 16, 1)
    Place-Names-3-12-Natural Open Sans Italic rgba(238, 255, 229, 1) rgba(32, 64, 16, 1)

    ⚫ Place

    Layer text-font text-color text-halo-color
    Place-Names-13-Place Roboto Regular rgba(255, 252, 252, 1) rgba(32, 32, 32, 1)
    Place-Names-3-12-Place Roboto Regular rgba(255, 252, 252, 1) rgba(32, 32, 32, 1)

Verification

  1. Stylesheet comparison between topographic.json and aerialhybrid.json where all three class types are visible:

    Topographic
    Aerial Hybrid (Before)
    Aerial Hybrid (After)
  2. Zoom level comparison of updated aerialhybrid.json stylesheet where backdrop imagery changes:

    🔵 Water

    Feature Zoom Level = 11 Zoom Level = 12
    Lake Wairarapa
    Lake Ellesmere

    🟢 Natural

    Feature Zoom Level = 11 Zoom Level = 12
    Cape Farewell
    Mount Taranaki

Copy link
Contributor

github-actions bot commented Oct 4, 2024

Vector Style Update

Vector Style updated for st_aerialhybrid

@tawera-manaena tawera-manaena changed the title Feat/update aerialhybrid style Feat/update aerialhybrid style BM-1085 Oct 4, 2024
@tawera-manaena tawera-manaena changed the title Feat/update aerialhybrid style BM-1085 feat(style): update aerialhybrid style BM-1085 Oct 4, 2024
@tawera-manaena tawera-manaena changed the title feat(style): update aerialhybrid style BM-1085 style(vector): update aerialhybrid style BM-1085 Oct 4, 2024
@dwsilk dwsilk added this pull request to the merge queue Oct 7, 2024
Merged via the queue into master with commit f18171c Oct 7, 2024
7 checks passed
@dwsilk dwsilk deleted the feat/update-aerialhybrid-style branch October 7, 2024 01:31
github-merge-queue bot pushed a commit that referenced this pull request Oct 7, 2024
### Motivation

As a Basemaps user, I want place names to be visually distinct based on
their type. I want to be able to easily identify different geographic
features based on their labels.

### Modifications

1. Deleted the following layers from the `labels.json` stylesheet:

   | Layers |
   | - |
   | `Place-Names` |
   | `Place-Names-3-12` |

2. Copied the following layers from the `aerialhybrid.json` stylesheet
into the `labels.json` stylesheet:

   | Layers |
   | - |
   | `Place-Names-13-Water` |
   | `Place-Names-13-Natural` |
   | `Place-Names-13-Place` |
   | `Place-Names-3-12-Water` |
   | `Place-Names-3-12-Natural` |
   | `Place-Names-3-12-Place` |

> These layers from the `aerialhybrid.json` stylesheet were styled in
#987

### Verification

1. Stylesheet comparison between the `Topographic` and `Aerial` layers
where all three class types are visible:

   | Topographic |
   | - |
   | ![][topographic] |

   | Aerial (Before) - Labels: On |
   | - |
   | ![][aerial_before] |

   | Aerial (After) - Labels: On |
   | - |
   | ![][aerial_after] |

[topographic]:
https://github.com/user-attachments/assets/e6af4291-3ad9-4a97-95b3-4cebe800ec29
[aerial_before]:
https://github.com/user-attachments/assets/86618577-ef5d-4166-a651-67b110c8c8df
[aerial_after]:
https://github.com/user-attachments/assets/1b6923c5-378c-4de1-89e9-d2d36bf1f225
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants