GrapesJS Issues

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

1180 issues found

#3618July 13, 2021by san-1231 answer
0 reactions

BUG: style manager size not setting to image control

WARNING READ and FOLLOW next 3 steps, then REMOVE them before posting the issue Follow CONTRIBUTING Guidelines https://github.com/artf/grapesjs/blob/master/CONTRIBUTING.md Do a quick SEARCH first, to see if someone else didn't open the same issue All relative statements/questions have to be filled/answered, otherwise,...

san-123

Its duplicate

#3615July 13, 2021by onmagic3 answers
0 reactions

BUG: text in text component cannot update , cannot get final result from toHTML

it is text component, if select body, the text will correct save/apply/store in component toHTML method, if just updating text, but not leave the text component, just click save then use "editor.getHTML()", the result is not updated.

v-pasha2

Did you get a solution on this ?

v-pasha2

We are facing below issue with version "0.17.19" where as on version "0.14.62" its was working fine. We were trying inline editing experience, So when we check the component.getEl() function it returns the updated text for component where...

v-pasha2

@artf could you please check on this or any-work around for this ?

#3614July 12, 2021by kuhelbeherNo answers
0 reactions

BUG: Background color can't be set to #000000

Hello! Found a bug with background-color. It cannot be set to #000000 via color-picker in styles. Here's a demo but you can check it yourself in the grapesjs demo: https://user-images.githubusercontent.com/37115006/125304571-62303e00-e336-11eb-9e5d-a5e9d3b1c7e2.mp4 Other colors work fine.

#3613July 12, 2021by anlumo1 answer
1 reactions

BUG: Inconsistent Image Selection

Version: 0.17.19 Are you able to reproduce the bug from the demo?[X] Yes[ ] No Reproducible here. What is the expected behavior? Selecting images in the asset manager interface by clicking on them. Describe the bug detailed What is the current behavior? When you change the asset associated with an image component, it'...

artf

Sounds legit, will be updated in the next release

#3612July 12, 2021by TRIGGEREDcoder3 answers
0 reactions

BUG: The grapesjs slider buttons, tabs are not working as expected

Version 0.17.19(grapesjs) Version 0.1.5(grapesjs-lory-slider) Version 1.0.6(grapesjs-tabs) Are you able to reproduce the bug from the demo? No What is the expected behavior? I should be able to traverse through different slides and tabs when clicked on previous or next buttons and tab names respectively. Describe the...

artf

Are you able to provide a reproducible demo, please?

TRIGGEREDcoder

Sorry @artf , I was facing this because of the script tag in the generated html which is of no use in angular component html. How can I use the html generated in angular files if the html contains script tag functions.

artf

@TRIGGEREDcoder if the main issue is solved I'm closing this one. Please open a new Discussion if you have questions

#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

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

#3579June 30, 2021by mcottret3 answers
2 reactions

BUG: RTE misplaced when opened right after dropping `activeOnRender` component

Version: 0.17.4 Are you able to reproduce the bug from the demo?[x] Yes[ ] No Steps to reproduce:Go to the demoDrop a "Text" block inside the canvasClick on the "Text" component that was just droppedThe opened RTE is misplaced (cf attached screenshot) What is the expected behavior? The opened RTE should be positioned...

artf

Yes, I guess you're right. It doesn't actually make sense activating RTE without selecting the component (it probably makes sense with all other "activatable" blocks).

gustavohleal

Hi. Me and @ronaldohoch have found a workaround for this problem. We trigger the scroll event when the RTE is enabled. Here's the code:

ronaldohoch

Hello, just updated the file from commit bb4a661, and i think it's so close, i'm using the grapesjs-ckeditor code and the first open of RTE, i'ts placed right: But at the second time the rte is open, it's misplaced again... :/ I'll keep ch...