Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

widgets in map wrongly align with a specific config #10676

Closed
1 task done
MV88 opened this issue Nov 13, 2024 · 1 comment · Fixed by #10681 or #10689
Closed
1 task done

widgets in map wrongly align with a specific config #10676

MV88 opened this issue Nov 13, 2024 · 1 comment · Fixed by #10681 or #10689

Comments

@MV88
Copy link
Contributor

MV88 commented Nov 13, 2024

Description

How to reproduce

  • use this config
{
        "name": "Widgets",
        "cfg": {
          "cols": { "md": 48, "xxs": 1 },
          "rowHeight": 10,
          "defaults": {
            "initialSize": {
              "h": 8,
              "w": 8
            }
          }
        }
      }
  • open a map with some widgets in single widget layout (resize window until it triggers)

Expected Result

to see widgets panel correctly
Current Result

it goes very below and the widget is not fully visible, partially hidden
this is due to the fact that rowHeight from configuration affect the baseHeight here
https://github.com/geosolutions-it/MapStore2/blob/master/web/client/plugins/Widgets.jsx#L146-L148

  • Not browser related
Browser info (use this site: https://www.whatsmybrowser.org/ for non expert users)
Browser Affected Version
Internet Explorer
Edge
Chrome
Firefox
Safari

Other useful information

@MV88 MV88 added the bug label Nov 13, 2024
@MV88 MV88 added this to the 2024.02.02 milestone Nov 13, 2024
@MV88 MV88 assigned MV88 and unassigned tdipisa Nov 13, 2024
@tdipisa tdipisa assigned rowheat02 and unassigned MV88 Nov 15, 2024
@rowheat02 rowheat02 linked a pull request Nov 15, 2024 that will close this issue
12 tasks
@offtherailz
Copy link
Member

offtherailz commented Nov 18, 2024

Here a context file to test the issue I created:

  • issue_10676-context.json
  • The context is available on dev server here
    For double check, this is a screenshot of the issue as I've understood it. Correct?
    Image

@ElenaGallo ElenaGallo added the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Nov 20, 2024
@ElenaGallo ElenaGallo self-assigned this Nov 20, 2024
@tdipisa tdipisa removed the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment