[Bug] Devices with a percentage width not supported

GrapesJS / grapesjs

Free and Open source Web Builder Framework. Next generation tool for building templates without coding

https://grapesjs.com

BSD 3-Clause "New" or "Revised" License

22.38k stars 4.06k forks source link

Closed dsgh closed 6 years ago

dsgh commented 6 years ago

version: 0.14.8

Use the grapesjs-blocks-bootstrap4 plugin.
Use the index.html example there - Keep the default devices (which include one defined as 100%)
Add any block with classes (eg Container)
Click that added block in the editor canvas.
This results in an error: Uncaught DOMException: Failed to execute 'querySelectorAll' on 'Element': '#gjs-css-rules-100%' is not a valid selector.

error

maxtacco commented 6 years ago

This is probably related to https://github.com/artf/grapesjs/issues/896. I guess the logic should be adjusted to handle different types of units, i.e px, em, % etc.

lock[bot] commented 5 years ago

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.