Issue #3463Opened May 18, 2021by anatoli-dp1 reactions

BUG: inspector overflow broken

Question

WARNING

READ and FOLLOW next 3 steps, then REMOVE them before posting the issue

  1. Follow CONTRIBUTING Guidelines https://github.com/artf/grapesjs/blob/master/CONTRIBUTING.md
  2. Do a quick SEARCH first, to see if someone else didn't open the same issue
  3. All relative statements/questions have to be filled/answered, otherwise, the issue might be CLOSED
  4. JSFiddle Starter template https://jsfiddle.net/szLp8h4n
  5. CodeSandbox Starter template https://codesandbox.io/s/1r0w2pk1vl

Version:

You can get the version by typing grapesjs.version into the console

Are you able to reproduce the bug from the demo?

  • [ x] Yes
  • No

What is the expected behavior? when stacking anything within each other the inspector should make appropriate room for the text of the component and its controls properly positioned

Describe the bug detailed when stacking items past the inspectors width the rows expand but no text is seen and controls are not properly positioned

What is the current behavior? as stated above (idk if i even explain it well but screenshots are provided)

Describe the bug detailed

Are you able to attach screenshots, screencasts or a live demo?

  • [ x] Yes (attach)
  • No Screenshot from 2021-05-18 09-50-01 Screenshot from 2021-05-18 09-50-07 Screenshot from 2021-05-18 09-50-15

Answers (3)

anatoli-dpMay 22, 20211 reactions

Maybe put the controls in the front of the text and then offset the entire box slightly for each indent or stacked item. That way u can stack them as far as you want without issue 🤷 Im going to end up rolling my own solution to achieve that but since I don't understand the structure of how ur code is done I'm not gonna be able to help much. But would be nice if it was that way by default

artfMay 22, 20210 reactions

Yeah, unfortunately, the default layer panel is not that scalable, we need to rethink how is rendered

artfMay 22, 20210 reactions

Yeah moving controls in front of the text sound reasonable (eg. and make them visible only when I hover the component). Anyway, being not such a common case having so much messy structure I wouldn't put a lot of priority on this problem, but for sure it would cool to fix such a limitation. If anyone is available for help, here you can find views of the Layer Manager: https://github.com/artf/grapesjs/blob/dev/src/navigator/view/ItemView.js

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.