Skip to content

Conversation

@arxitpln
Copy link
Contributor

Description

"enableDynamicLegend" option for ARCGIS layers.

image

Please check if the PR fulfills these requirements

What kind of change does this PR introduce? (check one with "x", remove the others)

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Other... Please describe:

Issue

What is the current behavior?
#11113

What is the new behavior?
Option visible in the "Display" tab option
Effect visible in the TOC

Breaking change

Does this PR introduce a breaking change? (check one with "x", remove the other)

  • Yes, and I documented them in migration notes
  • No

Other useful information

@arxitpln arxitpln mentioned this pull request Dec 18, 2025
12 tasks
@allyoucanmap
Copy link
Contributor

@arxitpln The PR is approved now. I included few changes:

  • 5498e19 Removed formatting from unrelated part of the file (the diff in github were highlighting a wider section as changed)
  • ffe5bb3 Review a the query legend workflow:
    • removed useEffect for legendNotVisible and replaced with useMemo
    • included cancel token and debounce with a a timeout to reduce the number of queryLegend requests
    • review visibility of error and empty messages
    • review unit tests

@allyoucanmap allyoucanmap linked an issue Jan 12, 2026 that may be closed by this pull request
2 tasks
@allyoucanmap allyoucanmap merged commit b03ba39 into geosolutions-it:master Jan 12, 2026
11 checks passed
@allyoucanmap
Copy link
Contributor

@ElenaGallo please test this enhancement on dev, thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dynamic Legend for ARCGIS layers

3 participants