#3618July 13, 2021by san-1231 answer
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,...
#3615July 13, 2021by onmagic3 answers
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
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
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
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
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
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
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
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
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...