#1606November 26, 2018by tranthanhhoa1 answer
How to reproduce it:Goto https://grapesjs.com/demo.htmlDrag a form in Forms category into canvasType something => focus out => focus in => hit delete key Result: Input disappear Expect: Input shouldn't disappear when user delete text
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.
#1605November 26, 2018by rajeshmaropost3 answers
hi @artf I've gone through this Drag and Drop Editor and what an amazing work. I also read your docs and I tried to extend rich text editor but i m stuck b/w some thing. As you can see screen shot below i've added different options for Rich text Editor I implemented "paste as plain text" i want to another option for p...
artf
Use insertHTML instead of insertText
rajeshmaropost
hi @artf Here is my code. rte.add('insertHTML', { name: 'insertHTML', icon: '<i class="fa fa-paste"></i>', attributes: {title: 'PasteAsPlainText', "data-tooltip": 'PasteAsPlainText'}, result: rte => { rte.exec('insertHTML', prompt("Because...
artf
insertHTML is the native way to insert HTML strings but probably each browser implements its own set of rules. Instead, check this https://stackoverflow.com/questions/25941559/is-there-a-way-to-keep-execcommandinserthtml-from-removing-attr...
#1604November 26, 2018by umairabid2 answers
Apparently options passed to grapjs.init() are not being deep cloned and same object is being mutated which was originally passed. This is creating a weird issue, I cannot do this Mutation can be seen by logging myOptions before init and the result is very weird, editor without any components <img width="1440" alt="sc...
artf
The option object is passed to the plugin as it is and there is no reason to clone it
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.
#1602November 22, 2018by arthuralmeidap3 answers
Hey @artf , How have you been? Quick question: a Panel has a property called visible. I'm trying to set it to false but nothing happens. I'm getting the panel after the GrapesJS is rendered like editor.Panels.getPanel('myId').set('visible', false); I was digging into the source code and I didn't find any use to the vi...
arthuralmeidap
created MR https://github.com/artf/grapesjs/pull/1603
arthuralmeidap
Oh, I didn't say but I want to hide/show the panel according to some condition. I know how I could get the panel html element itself and change the display but I was wondering about the visible property
artf
I was digging into the source code and I didn't find any use to the visible property for the Panel model. Yeap, you're right 😬 I'd expect to see a listener in PanelView Would you like to create a PR for such a case?
#1601November 22, 2018by Aitortilla221 answer
Hello. I was wondering if I could put the iframe inside a "</div class="col"/>" bootstrap. Now iframe is generated full screen but mi composition in the web is: "</div class="row"> </div class="col-lg-2"></div> </div class="col-lg-8">"TEMPLATE"</div> </div class="col-lg-2"></div> <//div>" obviously the template that I...
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.
#1599November 20, 2018by 00salmon3 answers
Version 0.14.40Click "Continue"Select "You have successfully signed up!" header (sqh-header)Drag sqh-header block between "Give $10 and Get $10" and text belowsqh-header appears visually above the Give $10 Header, but below the Give $10 Header in the layer managerDrag sqh-header beneath "Rewards Dashboard" back inside...
artf
Seems like the issue is due to different empty "textnode"s you have in your JSON. How do you obtain that structure?
00salmon
Hey @artf, Thanks for pointing that out. The JSON came from an internal example with more components and the missing bits must've been converted to empty textnodes. I've removed the empty textnodes in the jsfiddle, but still seem to be get...
artf
Please update the live demo with cleaned JSON
#1598November 20, 2018by kewilson2 answers
Hi, our SAST scanner detected the following, thought you might like to know. https://raw.githubusercontent.com/artf/grapesjs/dev/dist/grapes.js Code (Line #29702): this.setElement(this.createElement(.result(this, 'tagName'))); ClientDOMStored_XSS exists @ public/dist/grapes.js Severity: High CWE: 79 https://cwe.mitre....
artf
Thanks for the report @kewilson but that line is totally legit for the editor. The library itself shouldn't be "able to hurt", most of the vulnerabilities might be added during the integration
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.
#1597November 20, 2018by Abhisheknanda13444633 answers
hi @artf Is it possible to add resizable property on the frame of layer-manager as screen shot below i want to add dynamic resizable property for this frame. Thanks.
artf
You can create external resizable panels like in this example https://codepen.io/artf/full/MQpZPj/ or you can use any other kind of javascript library and make the panel resizable (once the editor is loaded)
Abhisheknanda1344463
@artf how can i achieve this for existing layer-manager in newsletter plugin. Should i need to add same classes as above on editor load . Please Let me know from where i can start. 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.
#1596November 19, 2018by 4URGB3 answers
Hi, to understand and learn how your preset is working, it will be much more easier if you can upload a uncompressed file of grapesjs-preset-webpage.min to the package - what do you mean? #
4URGB
its no issue, but an answear will be nice...or not
cjpollard
All the code is here. https://github.com/artf/grapesjs-preset-webpage
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.
#1595November 19, 2018by acxvgalanov2 answers
Add a possibility to limit uploaded image size. Maximum image size should be configurable in options for grapesjs.init.
artf
You should be already able to do that by implementing your own uploadFile method
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.