Skip to content

Hillside shading fix#17

Merged
hgosalia merged 2 commits into
mainfrom
hillside-shading-fix
May 20, 2026
Merged

Hillside shading fix#17
hgosalia merged 2 commits into
mainfrom
hillside-shading-fix

Conversation

@hgosalia
Copy link
Copy Markdown
Member

No description provided.

@github-actions
Copy link
Copy Markdown

📋 PR Summary

Title: Hillside shading fix
Author: @hgosalia
Draft: No
1 files changed: +44 additions, -10 deletions

Description

This PR introduces a fix for hillside shading in the map application, enhancing the user interface for better usability. The reset view button's styling has been adjusted to improve its visibility and interaction, changing its font size and padding to create a more user-friendly design. Additionally, the exaggeration control element has been modified to ensure consistent styling and functionality, including an updated layout for the slider and its associated value display. These changes aim to provide users with a clearer and more accessible way to manipulate hillside shading features on the map.

Files

  • js/map.js — Modified: initMap, _applyTerrainAndProjection

@github-actions
Copy link
Copy Markdown

📋 PR Summary

Title: Hillside shading fix
Author: @hgosalia
Draft: No
4 files changed: +114 additions, -23 deletions

Description

This PR introduces several enhancements to the map styling functionality, specifically adding a "Bright Map" option and improving hillside shading. The "Bright Map" style is now selectable from the style menu and can be activated using keyboard shortcuts, enhancing user experience by providing more visual options. Additionally, the code modifications ensure that unsupported layer types are filtered out to prevent validation errors in MapLibre, which improves rendering performance and stability. The changes also include adjustments to the hillside shading logic to ensure better visual representation in the selected map styles, particularly enhancing the user interface for better usability.

Files

  • css/styles.css — Updated .maplibregl-ctrl-bottom-right
  • index.html — +1/-0
  • js/demo.js — +15/-0
  • js/map.js — Modified: _patchStyleBoundaries, addPinLayers, applyTheme, initMap +2 more

@hgosalia hgosalia merged commit b84287a into main May 20, 2026
1 check 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.

1 participant