Skip to content

[photo & carto] Définition explicite de l'ordre des couches fix #67#68

Merged
vsasyan-ignf merged 1 commit intomainfrom
67-bug-ordre-couches
Feb 16, 2026
Merged

[photo & carto] Définition explicite de l'ordre des couches fix #67#68
vsasyan-ignf merged 1 commit intomainfrom
67-bug-ordre-couches

Conversation

@vsasyan-ignf
Copy link
Contributor

No description provided.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR implements explicit layer ordering to fix issue #67 by assigning zIndex values to different layer types in the mapping application. The changes ensure that raster layers, context vector layers, display vector layers, and the pin layer are properly stacked visually.

Changes:

  • Added explicit zIndex values to WMTS/raster layers (zIndex: 0) in both initial creation and dynamic updates
  • Implemented proper layer insertion logic using insertAt to maintain correct layer order when adding new WMTS layers
  • Set zIndex values for all vector layer types: context layers (100), display layers (300), and pin layer (999)

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
app/src/components/composable/getWMTS.js Added zIndex: 0 to raster layers and changed layer insertion from addLayer to insertAt with correct positioning logic
app/src/components/MapBox.vue Added comments explaining layer types and implemented zIndex assignment for all vector layers during map initialization

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@vsasyan-ignf vsasyan-ignf merged commit dc64b69 into main Feb 16, 2026
7 checks passed
@vsasyan-ignf vsasyan-ignf deleted the 67-bug-ordre-couches branch February 16, 2026 10:08
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