#2802May 26, 2020by meyerco1 answer
DescriptionThe bug is reproduced on the current demo What happens ? When switching selection quickly between controls , the editor get stucked . Steps to reproduce on the demo Select one controls and move quickly selection to another . Do this few times the editor will get stucked . The error on console
artf
Thanks for the report @meyerco probably found the main issue, should be fixed in the next release
#2800May 25, 2020by meyerco3 answers
DescriptionThe bug is reproduced on the current demo Expected behavior When a tooltip is removed its css is removed What happens instead ? When a tooltip is removed its css is not removed Steps to reproduce on the demo Clear the canvas Drag a tooltip Remove the tooltipClick on Import button on top toolbar , the toolti...
artf
Yeah, I think you're right, probably replacing with might fix the issue, would you like to check and create a PR? :)
meyerco
@artf Maybe this code to keep the code generic ?
artf
@meyerco 😅 yeah sure, sorry, it was a copy-paste from my try
#2790May 19, 2020by motazad3 answers
@artf When I click on the devices(tablet, mobile), the size of the canvas is not updated But it Worked in Version 0.15.x and lower.
cdespinosaagmx
I have the same issue
kuhelbeher
I had the same issue after grapejs update. In my case it turned out that I was using old version of css file: https://unpkg.com/[email protected]/dist/css/grapes.min.css I updated it to use version as grapesjs and it solved my problem. Try...
alitorki2651
I have the same issue @artf
#2778May 15, 2020by ashishtyd2 answers
Hi I was trying to use grapesjs on visualsource pages. Following is the init function - initEditor: function(){ var self = this; this.editor = grapesjs.init({ container: '#gjs', fromElement: true, height: '500px', width: 'auto', storageManager: false, panels: { defaults: [] }, blockManager: { appendTo: '#blocks', bloc...
artf
Create a reproducible demo please
no-response[bot]
This issue has been automatically closed because there has been no response to our request for more information from the original author. With only the information that is currently in the issue, we don't have enough information to take ac...
#2774May 14, 2020by vinceumo1 answer
Hi, I'm having some issue with mobile first approach. The canvas render the wrong style when setting different styles at multiple breakpoints. But the code generated is good, this issue only happen in the canvas. https://codesandbox.io/s/xenodochial-shockley-s02lp?file=/index.js:496-524Are you using the latest release...
artf
We should apply the same logic from https://github.com/artf/grapesjs/blob/201e1f9a61783080f7be885e9e0be9a40ecc16cc/src/codemanager/model/CssGenerator.js#L151-L162 here https://github.com/artf/grapesjs/blob/201e1f9a61783080f7be885e9e0be9a40...
#2773May 14, 2020by Light08151 answer
Minor bug: When switching video type component to html5 provider it always takes the src of the previous provider. Instead I would expect it to be cleared. This also has another side effect since it seems (without checking code) that you set the provider depending of the src. This is especially annoying when you save...
artf
It should be fixed in the next release, thanks for the report
#2771May 13, 2020by antman33513 answers
Hello, I've just updated to the lasted version 0.16.12 and have a bug when trying to move a component that has the property textable:1 into a text component. It works on the previous version 0.16.3. When i move the component I get an error in the console Uncaught TypeError: Cannot read property 'attributes' of undefin...
artf
I'm leaving here a working example of a textable component https://jsfiddle.net/artur_arseniev/0b9ox72g/
0099FF
Looks to have been introduced in 9c24108dadf2ded39e8805b6c4a5f00c626c468b. As pointed out by @antman3351 the model.getView().render(); call is the trouble maker. A little further down the call stack we attempt to update the view's attribut...
bogdanbs
My 2 ct to this. The problem lies on mixing asynchronous and synchronous code. The enabling of the rte: enable(view, rte) as well as the ComponentTextView.onActive and ComponentTextView.disableEditing are now, since the fix for issue #3475...
#2766May 12, 2020by svalenciano813 answers
Hi Where can I reproduce it? In demo url: https://grapesjs.com/demo-newsletter-editor.html Browser info: Firefox 75.0 How can I reproduce it? Select a table or table-cell and try change background-color with picker selector. The result no change the color :(
svalenciano81
but, why alpha channel is "0" by default in Firefox and is "100" in chrome?
svalenciano81
Increase the alpha channel solve the problem :-)
artf
I close this in favor of #3260
#2765May 12, 2020by svalenciano813 answers
Hi, in grapesjs in Firefox 75.0 when "backspace key" (event.which === 8) is pressed the navigator is going back and grapesjs is lost. To reproduce it: Open Firefox browser Open URL: https://grapesjs.com/ Click on: "Newsletter Demo" buttom Delete a element from editor usign "backspace key" (NOT using supr key) The beha...
artf
Should be fixed in the next release
artf
@juice77 the issue was fixed in the latest release https://github.com/artf/grapesjs/compare/v0.16.17...v0.16.18
tiptronic85
Yes this is very annoying
#2758May 7, 2020by mattdeigh3 answers
This is reproducible on the demo site running . In the console, run and try to delete a component. It does delete the component, and I don't believe it should. It seems like the other Keymaps are unbinding fine. I'm not sure if this is an issue with grapesjs or keymaster. Noticed this when I was adding a custom keymap...
artf
I'm wondering if the unbind command in keymaster was meant to handle an individual key instead of multiple ones in the case of backspace but the copy/paste/undo/redo are working correctly. Well, it works with other keymaps so I'd say it ha...
mattdeigh
Man... I looked through Keymaster and that unbind function handles multiple keys, so the issue must be somewhere else. I didn't notice anything glaring when I looked. No errors show up when I try to unbind the key. I'm using GrapesJS in a...
artf
Are you good with the string split approach? Sure, let's see if it works