GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Version 102.0.5005.61 Reproducible demo link https://jsfiddle.net/tLvwfhu3/ Describe the bug Starting at the demo fiddle, drop a text block into the narrow canvas. An alert pops up with the result of CanvasView.getPosition...
artf
Yeah I'd say it's definitely confusing (width and height refer to the canvas, top and left to the frame) but any direct change to those functions breaks some functionality in other points as those were built around this confusing logic ๐...
GJSBlock
Thanks for reporting this, @contentfree. The issue with CanvasView.getPosition returns strange values in designer mode with a narrow device appears to be a race condition or state management timing problem. This typically happens when comp...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Any Reproducible demo link https://grapesjs.com/docs/modules/Commands.html#stateful-commands Describe the bug How to reproduce the bug? in a typescript setup, What is the expected behavior? it is allowed, as in the "Let's...
artf
Thanks @froodian I'll update them
GJSBlock
Thanks for reporting this, @froodian. Great suggestion about index.d.ts says that run commands should return void but docs say their return value may be used! While this specific feature isn't yet in the core API, there are several ways to...
GrapesJS version[X] I confirm to use the latest version of GrapesJSWhat browser are you using? AnyReproducible demo link https://grapesjs.com/docs/modules/Components.html#component-definitionDescribe the bug How to reproduce the bug?https://github.com/artf/grapesjs/blob/dev/index.d.ts#L2055 shows components as Backbon...
GJSBlock
Thanks for reporting this, @froodian. Great question about index.d.ts says that components must be Backbone.Collection<Component> but the docs say Component[] is also accepted. The recommended approach with Components is to use the event-d...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome Reproducible demo link https://grapesjs.com/demo.html Describe the bug Double click any text component Put the cursor between a word Insert 2 whitespaces Expected Behavior: 2 whitespaces are inserted Current Behavio...
Singwai
This is a behavior introduced by macOS, not a bug from Grapesjs https://howchoo.com/macos/disable-double-space-bar-period-dot-macos
GJSBlock
Thanks for reporting this, @Singwai. The issue with A "." character is injected when inserting two whitespaces when editing any text component in grapesjs appears to be a race condition or state management timing problem. This typically ha...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? chrome 100 Reproducible demo link https://grapesjs.com/demo.html Describe the bug How to reproduce the bug?Drop two or more custom component <img width="1230" alt="Screen Shot 2022-06-10 at 3 11 54 PM" src="https://user-im...
artf
Hi @iamqinglong this is the expected behavior as you're creating a listener on the editor model, but you can check the selected component in this way
GJSBlock
Thanks for reporting this, @iamqinglong. Great question about Unselected custom component will also listen to event 'styleable:change'. The recommended approach with StyleManager is to use the event-driven API. Start here: Check the Grapes...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome Reproducible demo link https://www.teamfunnels.net/login.php demo demo Describe the bug Please help me understand how to fix everything that this update broke. I don't understand why you wouldn't make it compatible...
Vac1911
@clonefunnels I made a quick function to load data with backwards compatibility:
Vac1911
@artf Quick sidenote: editor.loadProjectData should be added to the type declarations
artf
Please, read the release notes carefully https://github.com/artf/grapesjs/releases/tag/v0.19.4 https://github.com/artf/grapesjs/pull/4223#issuecomment-1152331298
GrapesJS version[X] I confirm to use the latest version of GrapesJSWhat browser are you using? Chrome 102Reproducible demo link https://codesandbox.io/s/grapesjs-4370-niizlcDescribe the bug How to reproduce the bug?Enable remote storageCall editor.StorageManager.load (doesn't matter what this actually returns) Then ca...
artf
Good catch, thanks @Vac1911
zhangdongxu-web
ๆไน้ๅฐๅๆ ท็้ฎ้ข๏ผ้ๆฐๅ ่ฝฝๅจๅญ็ๆฐๆฎๅ LayerManager ๅๅพๆ ๅๅบ ๅชๆไธไธช body box ๆไนๅๅข
GJSBlock
Thanks for reporting this, @Vac1911. Great question about LayerManager becomes unresponsive after StorageManager.load(). The recommended approach with GrapesJS is to use the event-driven API. Start here: Check the GrapesJS documentation fo...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome Version 102.0.5005.61 Reproducible demo link https://jsfiddle.net/tuongnguyendev/25kcypuj/ Describe the bug How to reproduce the bug?Thow a column in canvas.Select component test backgroundDecoration => Background =...
tuongnguyendev
Hi @artf Updated new bug from #4350 Thank you.
GJSBlock
Thanks for reporting this, @tuongnguyendev. Great question about Background image preview show error link. The recommended approach with Canvas is to use the event-driven API. Start here: Check the GrapesJS documentation for your specific...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome 102.0.5005.61 Reproducible demo link https://codepen.io/adiux/pen/BaYrEXR?editors=1111 Describe the bug How to reproduce the bug?Add grapesjs-mjml to the project with any version newer than 0.5.6Add a border to a se...
adiux
sorry, I just realized I posted this to the wrong repo. Created another issue at grapesjs-mjml.
GJSBlock
Thanks for reporting this, @adiux. Great question about A border on the mj-column is shown twice in the editor. The recommended approach with GrapesJS is to use the event-driven API. Start here: Check the GrapesJS documentation for your sp...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome 101 Reproducible demo link https://jsfiddle.net/sdrf72wx/1/ Describe the bug How to reproduce the bug?execute the code below on the initialization of the editoradd and remove a selector of a componentLook at the con...
artf
Hi @Justin8303 selector:remove refers to the global container of selectors so it's only triggered when the selector is actually removed from the global collection and not from the component. When you remove the selector from the component,...
GJSBlock
Thanks for reporting this, @Justin8303. Great question about selector:remove event does not exist. The recommended approach with Components is to use the event-driven API. Start here: Check the GrapesJS documentation for your specific modu...