GrapesJS Issues

Search parsed GitHub issues, see highlighted answers, and open the original discussion when needed.

3464 issues found

#3607July 9, 2021by gMakaanNo answers
0 reactions

FEAT: code viewer - go to selected canvas block(s)

When view code is clicked and we have selected element(s) in the canvas, code viewer can show it's corresponding code and select it outerHTML - great feature for instant copy from there or fast inspection.

#3606July 9, 2021by Palash-Mandal1 answer
0 reactions

BUG: if I use float from alignment float not clear that's why design issue occur.

Hello @artf , There a generic issue when try to create a layout using float form alignment. The issue is float not clear automatically. how do I add new class in row when user set flow value float left /right <div data-gjs-type="default" draggable="true" data-highlightable="1" class="row" id="iroj"><div data-gjs-type=...

artf

Please follow the issue template

#3605July 9, 2021by aryanchopraNo answers
0 reactions

QUESTION: Adding selected component style to a block

So I'm trying to fetch the selected component and make a block out of it, however, I'm struggling with adding the css of the same. my approach looks like this currently: editor.BlockManager.add(widgetname, { label: widget name, content: editor.getSelectedAll(), category: panelname, attributes: { title: widgetname, cla...

#3604July 9, 2021by NoumanAhmad-Braqsol2 answers
0 reactions

editor.on('component:update:content', model => { }}) not work in some cause

if the user remove all text from a element and click some where else . then again try to select empty element he is not able to select it again because it is hide and i dont know where it goes. So for this issue I am using this code this code set a space in text if user remove all text from a element . By doing this t...

artf

As already suggested here https://github.com/artf/grapesjs/issues/3603#issuecomment-885305546 avoid doing what you're doing now and add a simple default CSS in case the element is empty

NoumanAhmad-Braqsol

Thank you i got it !

#3603July 8, 2021by NoumanAhmad-Braqsol1 answer
0 reactions

Issue in Text Edit . Text not select agian if it remove

First of all thanks for the great tool @artf . As i am working on a project . So i have an issue while text editing . if i select a text box and edit a text it works fine' but if i remove all text and click somewhere else and again try to select text box . it not work because empty text go hide and did not select agai...

artf

I'd say more a layout issue. As the box doesn't have any text it loses its dimensions so it's hard to enable it again. Anyway, you can extend your text components and add some extra styles to make it easier to select when are empty. For th...

#3600July 7, 2021by sathyanarayananaCESNo answers
0 reactions

[Question]: Event listener to visibility(eye) icon in the Layer Manager

I would like to listen to the eye icon in the layer manager from the editor, based on which, I need to write the custom styles. Is there any way we can do that? I got the code, from another issue, I would like to extend the toggleVisibility method from here. Is it possible? if so, how can we do that? Secondly, I would...

#3599July 7, 2021by niranjank1511 answer
0 reactions

@keyframes not adding even after i pass the the keyframes object

Hi @artf, I have added the let cc = editor.CssComposer; cc.setRule('.fadetop-animate', { opacity: 0, 'animation-name': 'fadeTop' }, { atRuleType: 'keyframes', atRuleParams: 'fadeTop', selector: 'to', style:{ opacity: 1 } }); as per the Document [](https://grapesjs.com/docs/api/css_composer.html#setrule) however rule f...

artf

Use this one

#3596July 6, 2021by aimeos3 answers
0 reactions

BUG: Values are not escaped

GrapesJS code is open to XSS issues because values are inserted into DOM without escaping, e.g. https://github.com/artf/grapesjs/blob/dev/src/assetmanager/view/AssetImageView.js#L30 If model.getFilename() returns <img src=x onerror=alert(document.cookie)>.jpg, this can result in an account takeover. Instead the code s...

aimeos

@artf What are your plans because the vulnerability of GrapesJS to XSS is a pretty big security problem

artf

Yeah thanks for the report @aimeos I'll fix it in the next version for sure

aimeos

@artf The documentation also uses ${var} to insert variables into templates often. This should also be changed to avoid that developers introduce security issues too without knowing that.

#3595July 6, 2021by Palash-MandalNo answers
0 reactions

BUG:

Hello @artf , It's a generic issue the width, height property value set to auto while change view to mobile or tablet view. But the auto property not reflects in in CSS that's why desktop value is in there and issue is arise. If I change the value of width to auto and press tab / enter the value still not reflects in...

#3594July 6, 2021by maxtsh2 answers
1 reactions

BUG: Hover on component when parent of editor scrolls, show wrong positioning

Version: 0.17.19 Are you able to reproduce the bug from the demo? No, because its full screen and there is no parent container scroll. What is the expected behavior? Hover state cadre to see the component changes position Describe the bug detailed When I want to select or hover on specific component, it shows wrong po...

artf

If the scrollable element is not the editor's parent node, you might need this option https://github.com/artf/grapesjs/blob/dc45604962e4e6a4a075f19d5de67090bac15fa5/src/editor/config/config.js#L157-L163

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...