#1422September 13, 2018by FrciSmrci2 answers
Hey @artf, as already mentioned in this closed ticket => I think that this specific enhancement broke the behaviour of properties without units, like Opacity. The slider behaves weirdly in this case, even on the GrapesJS demo. Thank you in advance.
artf
You're right, I think I should use parseFloat here, will be fixed in the next release
lock[bot]
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.
#1421September 12, 2018by Valodya3 answers
Hello I assume there is a bug with class when Panel Button is disabled https://github.com/artf/grapesjs/blob/dev/src/panels/view/ButtonView.js#L22
artf
Yeap, the fix is ready for the next release. Thanks
Valodya
thanks @artf for the answer, do you have any estimate when it will be released?
artf
Probably during next week
#1418September 11, 2018by ageir3 answers
Running latest version (0.14.29) on local server. Using the example webpage preset. Ubuntu 18.04, Chromium 68. Assetmanager init: And extend image type: The all_files variable is load from php, and it has the right data and format, all images stored on the server. The problem is that the onRemove is not triggering as...
artf
Hi Göranbut reappearing when reloading the page you're initializing the asset manager with assets: allfiles so if you don't change allfiles you always get the same set of imagesAlso when dragging an image to the canvas, the image is not sh...
ageir
@artf I don't think you understood me. They are reappearing because they didn't get deleted on the server. all_files are generated via php on page reload, and showing the correct files on the server. The problem is that onRemove isn't trig...
artf
Ah ok probably because your assets are already rendered. Try to apply those changes but inside a plugin
#1416September 11, 2018by dsgh3 answers
I have my own window.onbeforeunload handler, and I'm not passing in a noticeOnUnload config to grapesjs, so I wouldn't expect grapesjs to mess with it. PR for this incoming.
artf
Thanks Duarte, fix applied and will be available in the next releaseI'm not passing in a noticeOnUnload config to grapesjs, so I wouldn't expect grapesjs to mess with it. Just a note, noticeOnUnload is true by default 😬
dsgh
Oops, I meant I'm passing a noticeOnUnload as false 😁 Thanks!
lock[bot]
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.
#1411September 10, 2018by zurp2 answers
Questions Q: Are you using the latest release (older versions are NOT supported)? A: Yes Q. Are you facing the bug with your local copy of GrapesJS or with the current demo? A: Local copy, latest chrome on Linux 64bitThe issue When opening the asset manager via editor.runCommand('open-assets') it should render the lis...
artf
The issue is that currently open-assets doesn't handle correctly the default types option (which I think, in this case, it should return all assets, even not images). I'll fix this in the next release, but for now, what I think it should b...
lock[bot]
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.
#1400September 3, 2018by FlashPapa2 answers
With /! grapesjs - 0.9.15 / When we edit a text box and we put a double dash and we use the function 'mjml2htm' we have the following error : Error: [MJMLError] ParseError: Error while parsing the file grapesjs-mjml.min.js:2:30405 a http://127.0.0.1/earcGrapesJs/app/bowercomponents/grapesjs-mjml/dist/grapesjs-mjml.min...
artf
@FlashPapa your grapesjs version is outdated and this is the core repository, which has nothing to do with mjml plugin
jmchaves
Double dash is still happening:
#1396September 1, 2018by jmrgz2 answers
Bug: when your pre-defined block or saved templates have two or more background layers, if you add a new layer, a lot of new empty layers are created. How to reproduce it:Go to https://grapesjs.com/demo.htmlSelect the pink Header blockGo to Decorations -> background and add a new layerDeselect the pink Header and sele...
artf
Thanks @xamdam777 for the report. Actually, as you already indicated, this happens when you import the style (and browser's CSS parsers are pretty messy), and unfortunately, in few cases, conflicts happen. (eg. Chrome generates a style tha...
jmrgz
Thank you very much @artf, now I completely understand the problem, damned browser's parsers! :) I have been doing some tests in addLayer() and refreshLayers(). This is my current approach, modifying refreshLayers() function, I try to dete...
#1394August 30, 2018by nojacko2 answers
https://grapesjs.com/demo.htmlIn the double click into the text "All text blocks could..." to trigger editing mode.Paste this text in: The text is not editable until Grapes is reloaded
artf
Thanks @nojacko found the issue, will be fixed in the next release
lock[bot]
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.
#1391August 29, 2018by FrciSmrci3 answers
Hey @artf , thanks for the enhancement! The implementation works as expected with some properties (font-size, border-radius, ...), but not with others (line-height, border-width, ...). I created a example/demo where it's visible that the configuration is the same in both scenarios, however the unit doesn't apply in th...
artf
I created a example/demo where it's visible that the configuration is the same in both scenarios Actually, you didn't indicate the defaults, where in case of line-height takes normal by default and so resets its unit on initialization
FrciSmrci
@artf I think that this specific fix messed up the properties without units, like Opacity. The slider behaves weirdly in this case, even on the demo.
lock[bot]
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.
#1373August 21, 2018by dsepulved2 answers
Hi, I am using backbone-deferred https://github.com/arhea/backbone-deferred, when starting GrapesJS this library is deleted, I have to restart the application to get it back, any way to solve it please?
artf
I don't know, try to load that library after GrapesJS or maybe check this http://backbonejs.org/#Utility-Backbone-noConflict it might be an issue as grapesjs, once compiled, contains its own version of Backbone
lock[bot]
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.