Skip to content

Fix #10513 Print preview state is not aligned with TOC#10517

Merged
allyoucanmap merged 2 commits intogeosolutions-it:masterfrom
rowheat02:fix/printlayersalignwithtoc
Aug 27, 2024
Merged

Fix #10513 Print preview state is not aligned with TOC#10517
allyoucanmap merged 2 commits intogeosolutions-it:masterfrom
rowheat02:fix/printlayersalignwithtoc

Conversation

@rowheat02
Copy link
Copy Markdown
Contributor

Description

This PR fixes the issue #10513. Before layers were only filtered by it's visibility, now layers are also filtered considering it's group visibility.

Please check if the PR fulfills these requirements

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

  • Bugfix

Issue

What is the current behavior?

#10513
Layer in the map preview of print plugin is visible even though it's group is invisible. In TOC it is working fine.

What is the new behavior?

Like in TOC now in print preview map also layer will be visible if it's group is visible.

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

@rowheat02 rowheat02 requested a review from allyoucanmap August 23, 2024 08:43
@rowheat02 rowheat02 self-assigned this Aug 23, 2024
@rowheat02 rowheat02 changed the title Fix/printlayersalignwithtoc Fix #10513 Print preview state is not aligned with TOC Aug 23, 2024
@rowheat02 rowheat02 added this to the 2024.02.00 milestone Aug 23, 2024
@rowheat02 rowheat02 linked an issue Aug 23, 2024 that may be closed by this pull request
1 task
@rowheat02 rowheat02 added bug investigation Internal estimate needed An estimation\ plan needs to be created for this issue. labels Aug 23, 2024
@tdipisa tdipisa removed the estimate needed An estimation\ plan needs to be created for this issue. label Aug 23, 2024
@allyoucanmap allyoucanmap merged commit 04e8784 into geosolutions-it:master Aug 27, 2024
@allyoucanmap
Copy link
Copy Markdown
Contributor

@ElenaGallo please test this fix on dev, thanks

tdipisa pushed a commit that referenced this pull request Aug 14, 2025
Co-authored-by: RowHeat <40065760+rowheat02@users.noreply.github.com>
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.

Print preview state is not aligned with TOC

3 participants