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

Inconsistency in width display of "Map" module on Landing Pages in GrapeJS #13574

Open
1 task done
code5rick opened this issue Mar 22, 2024 · 3 comments
Open
1 task done
Labels
bug Issues or PR's relating to bugs needs-triage For new issues/PRs that need to be triaged

Comments

@code5rick
Copy link
Contributor

code5rick commented Mar 22, 2024

Mautic Version

5.0.x series

Way of installing

I downloaded a release from https://www.mautic.org/mautic-releases

PHP version

8.0.2

What browsers are you seeing the problem on?

Chrome

What happened?

When utilizing the GrapeJS builder for landing pages, we've encountered an issue with the "Map" module. Despite appearing to have a 100% width when inserted onto the page, it is actually displaying with a fixed width of 350px upon saving.

This discrepancy between the displayed width and the actual width could potentially mislead users into believing that the module spans the full width of the page when, in fact, it does not.
image

First reported on: #13283 (comment)

How can we reproduce this issue?

Step 1: Go to Components > Landing Pages.
Step 2: Create a new landing page.
Step 3: Provide a title for the landing page.
Step 4: Choose the "Blank" theme, then click on "Builder."
Step 5: Add the "Map" module to the landing page. Notice that it initially appears to occupy the full width of the page.
image

Step 6: Exit the builder by clicking on the close button, then select "Save & Close."
Step 7: Preview the page to visualise the actual width of the "Map" module.
image

Relevant log output

None.

Code of Conduct

  • I confirm that I have read and agree to follow this project's Code of Conduct




Care about this issue? Want to get it resolved sooner? If you are a member of Mautic, you can add some funds to the Bounties Project so that the person who completes this task can claim those funds once it is merged by a member of the core team! Read the docs here.

@code5rick code5rick added bug Issues or PR's relating to bugs needs-triage For new issues/PRs that need to be triaged labels Mar 22, 2024
@dsp76
Copy link

dsp76 commented Apr 20, 2024

Hi @code5rick,
actually - step 5 is due to the missing column element? When I placed a column first, I didn't experience the full width in the editor window.

image

As a workaround, you could add the 100% for width and set the max width to the absolut width of your container.
Not sure if this is a Mautic or Grape-JS Bug.

@RCheesley
Copy link
Sponsor Member

@code5rick any thoughts on this?

@code5rick
Copy link
Contributor Author

I didn't put the module inside of a column or a section. Maybe thats the problem?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issues or PR's relating to bugs needs-triage For new issues/PRs that need to be triaged
Projects
None yet
Development

No branches or pull requests

3 participants