Issue #5705Opened February 26, 2024by bernesto0 reactions

BUG: Component tools misalign when the editor is resized

Question

GrapesJS version

  • I confirm to use the latest version of GrapesJS

What browser are you using?

Chrome v122

Reproducible demo link

https://jsfiddle.net/Lk2bwhp4/

Describe the bug

How to reproduce the bug?

  1. Place the editor inside of a container, then programmatically resize the container's width.

What is the expected behavior? The editor and all of its children should resize accordingly.

What is the current behavior? All component's canvas spots in the editor retain their offsets from prior to the canvas resize.

  • Note, a window resize of resets the positions. This must be done programmatically in the window. Use cases would be where the editor is on a page with other panels that may resize the available space for the editor.
  • In the example below, view how the tools align to the related element after resizing the editor to a larger width. The tools are left at the old location until an event triggers an update. A window resize or preview toggle will update the view.

image

Code of Conduct

  • I agree to follow this project's Code of Conduct

Answers (3)

artfMarch 4, 20240 reactions

Thanks @bernesto for the report but as we can't detect programmatic layout changes without impacting the performances, in this case, you have to update the editor layout manually via editor.refresh() (eg. in your example you would put the call right at the end of your resizeEditor function)

bernestoMarch 4, 20240 reactions

Hi @artf, I actually had this accidentally mixed in the last pull request you accepted (src/canvas/model/Frame.ts). You might want to take a look at what I added and see if it is causing an issue. We have it in testing right now and haven't seen a degradation in performance and it addresses the issue.

artfMarch 7, 20240 reactions

I actually had this accidentally mixed in the last pull request you accepted

and I actually removed it cause it seemed to be unrelated 🤣

Not a problem to add it back but consider that your example will work only because of the desktop Device. The width is filling the container and that triggers the iframe window resize (which then will trigger the added event). It won't work in case you change to a smaller Device, in that case no resize event will be triggered so you still need the refresh call.

Related Questions and Answers

Continue research with similar issue discussions.

Paid Plugins That Match This Issue

Curated by issue keywords and label relevance to help you ship faster.

View all plugins

Loading paid plugin recommendations...

Browse Plugin Categories

Jump directly to plugin category pages on the marketplace.