Are you using the latest release (older versions are NOT supported)? YES, 0.15.9Are you facing the bug with your local copy of GrapesJS or with the current demo?Local copyAre you able to reproduce the bug from the demo?No, not React app.What is the expected behavior?Toolbar buttons should only be added once on load of...
artf
Ok, seems like we don't clear the RTE instance in the destroy method... I'll fix it for the next release
simplecommerce
For now, I have to do rte.remove before my rte.add so it doesn't duplicate.
Steps to reproduce:Select all text in a componentMove the componentSelect all text againMove the component againDeselect the component Expected result: Text remains Actual result: Text disappears Reproducible in a web page demo, version 0.15.9. See attached screencast:
Cannot edit text components properly if a zoom value is applied on the canvas. GrapesJS version:0.15.8 The issue can be reproduced in the demo as well: https://jsfiddle.net/1n9ruwa4/ Expected behavior: Zoom is applied on canvas using:editor.Canvas.setZoom() The text components can be still edited and 'space'(Keycode:3...
mattdeigh
Ahhh, I see it now. Wasn't tracking with the code completely. I'll play around with that and let you know what I come up with.
Sphenox
I'm writing on this issue, since you can't add a 'space', if you are focused on an <a> or a <button>. It is reproducable in the demo, if you add a Link: https://jsfiddle.net/1n9ruwa4/ Reason is because of the notTextable config from the ca...
Expected behaviour: When changing existing link with target = "blank" to target ="", the change propagates to html. Actual behaviour: When changing existing link with target = "blank" to target ="", the change does not propagate to html. Steps to reproduce:Save a page with link with target "New window" ("_blank").Clos...
Hi guys, I'm working with grapesjs which version is '0.15.8', and in 'absolute' mode when I drag a block into canvas, I find the styles of component are missing. Define component as following: And I found the code in Droppable here: /utils/Droppable.js@L78 the component's styles are replaced. Should 'addStyle' be used...
pouyamiralayi
@artf yes! this is not happening if i remove `script`. the behavior is fine.
pouyamiralayi
Hi @sheldonwr this bug you are referring to is not reproducible in my side, here i tried your type definition on the demo and it is working, please check the below code and see what is missing in your side: the issue i observed with this c...
artf
Ok, actually a weird thing happens when a component has script property defined... Thanks for the report guys, I'll check this out. Can you confirm that without script it behaves correctly?
Explanation Using https://grapesjs.com/demo.html v0.15.8 on Chrome 77.0.3~ Windows 10 Pro Build 18362 Importing a Video component with any Boolean attribute (controls, loop, autoplay) using the import modal will break all boolean attributes on the component until the component is deleted. Reproduction StepsNavigate to...
artf
At first look the issue might be here: https://github.com/artf/grapesjs/blob/a850a305b80b8f86c0e7e4fb92b6492c2e5a1957/src/dom_components/model/ComponentVideo.js#L115-L117 indeed in the export/import I'd expect just ... controls> and not .....
Hi, I can not edit label in Firefox browser (v. 69.0.3). The issue occurs when I double click the component to edit it. RTE shows toolbar but changing text or attributes is unable. The web console does not show any errors. When the text is selected and I am trying to for example bold the text then in web console there...
artf
https://bugzilla.mozilla.org/show_bug.cgi?id=853519 Probably the easiest solution would be to force the View to use span tag instead of label
schmelik
This was an interesting fix but it comes with issues... spans != labels when applying padding to the labels such as bootstrap labels the padding will not be applied to a span not unless display: inline-block; is also applied to the span
This can replicated in the demo: https://grapesjs.com/demo.htmlDrag in a text nodeDouble click to activate the RTEDrag the text component into another positionSelect another component The component is removed and the following error is logged to the console:
Hi ! Little problem when I type a style property with important. When I clear the value of the field (like color type for exemple) value is set to "undefined !important". For fix this, on Property.getFullValue, you must check the value before add " !important" : if (value !== undefined && this.get('important')) { valu...
artf
Thanks @WebEtSolutions it'll be fixed in the next release