Skip to content

Improve layout in the map viewer#11789

Merged
allyoucanmap merged 9 commits intogeosolutions-it:masterfrom
subashtiwari1010:feat/change-map-layout
Jan 30, 2026
Merged

Improve layout in the map viewer#11789
allyoucanmap merged 9 commits intogeosolutions-it:masterfrom
subashtiwari1010:feat/change-map-layout

Conversation

@subashtiwari1010
Copy link
Copy Markdown
Contributor

@subashtiwari1010 subashtiwari1010 commented Dec 10, 2025

Description

This PR updates the layout of the map viewer with the possibility of adding the map as the background and letting all the components act as the overlay over the map. The feature grid is added in the bottom container of the map layout.

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?
The feature grid is in the dock container of the map viewer.
#11794

What is the new behavior?
The feature grid is displayed in the bottom container of the map layout.

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

Copy link
Copy Markdown
Contributor

@allyoucanmap allyoucanmap left a comment

Choose a reason for hiding this comment

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

In addition to the inline comments:

Comment thread web/client/components/layout/MapViewerLayout.jsx Outdated
Comment thread web/client/components/layout/MapViewerLayout.jsx
Comment thread web/client/product/assets/css/viewer.css Outdated
Comment thread web/client/reducers/featuregrid.js Outdated
Comment thread web/client/plugins/featuregrid/hoc/withResize.jsx Outdated
Comment thread web/client/plugins/featuregrid/hoc/withResize.jsx
Comment thread web/client/components/layout/MapViewerLayout.jsx Outdated
Separate the js and css for the withResize hoc

Remove the dockSize implementation for feature frid and remove it logic from updated map layout epic
@cla-bot cla-bot Bot added the CLA Ready label Jan 27, 2026
@subashtiwari1010 subashtiwari1010 force-pushed the feat/change-map-layout branch 2 times, most recently from 477465b to b98cc06 Compare January 27, 2026 12:33
@allyoucanmap allyoucanmap merged commit 6b9fea2 into geosolutions-it:master Jan 30, 2026
12 checks passed
@allyoucanmap
Copy link
Copy Markdown
Contributor

@ElenaGallo please test this enhancement on dev, thanks

@tdipisa
Copy link
Copy Markdown
Member

tdipisa commented Jan 30, 2026

@subashtiwari1010
it seems there are a few inconsistencies I've noticed.

  1. Open both toc and catalog, then close catalog
image
  1. Open a map the first time then open catalog
image

Reproduced with: https://dev-mapstore.geosolutionsgroup.com/mapstore/#/viewer/60692

@subashtiwari1010
Copy link
Copy Markdown
Contributor Author

subashtiwari1010 commented Feb 5, 2026

Few inconsistencies that I've noticed:

  • Export panel
    There is some style issue with the header, footer for the Export panel
image
  • Import panel
    There is some style issue with the header, footer for the import panel
image
  • Resizable / movable popup -> Share modal
    When such popup is moved to the header or footer, it goes under them.
image image

@allyoucanmap
Copy link
Copy Markdown
Contributor

@subashtiwari1010 just noticed the second step of context creation is not showing map

image

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.

Improve the map viewer layout to use different container position

3 participants