#6047August 7, 2024by danstarnsNo answers
The process for publishing the build artifacts of the GrapesJS repository has had the following issues: Artifacts are expected to come directly from developers' machines. - https://github.com/GrapesJS/grapesjs/pull/6041 The process is undocumented We have multiple packages to release, and proper documentation is also...
#6046August 7, 2024by danstarns1 answer
The current codebase relies on commit hooks to enforce certain rules and checks. This ticket aims to: Remove existing commit hooks. Configure CI runners to perform these checks and report issues on the development branch or pull requests. Shifting these responsibilities to the CI runners will streamline the developmen...
danstarns
Related: https://github.com/GrapesJS/grapesjs/issues/6045
#6045August 7, 2024by danstarnsNo answers
The codebase currently uses outdated linting tools and formatting libraries. This ticket aims to: Upgrade to the latest versions of linting tools and formatting libraries. Ensure consistent linting and formatting rules across all projects. Integrate these tools into the development and CI pipelines for automated enfor...
#6044August 7, 2024by danstarns1 answer
Across the codebase, we have built and linked dependencies against very old versions of Node.js. See: https://github.com/GrapesJS/grapesjs/blob/dev/.github/workflows/publish.yml#L15 https://github.com/GrapesJS/grapesjs/blob/dev/.github/workflows/build.yml#L15 https://github.com/GrapesJS/grapesjs/discussions/5973 https...
danstarns
Related:https://github.com/GrapesJS/grapesjs/issues/6048https://github.com/GrapesJS/grapesjs/issues/6049
#6040August 6, 2024by Vampire-V3 answers
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Version 127.0.6533.89 (Official Build) (64-bit) Reproducible demo link Describe the bug Please check this.selected has a value of undefined . I used it in react and got an error exception. /*Returns model of the selected c...
artf
Hey @Vampire-V please follow properly the bug issue template. We can't help you here without a reproducible demo.
Aravinth-2020
### GrapesJS version[x] I confirm to use the latest version of GrapesJS### What browser are you using?Version 127.0.6533.89 (Official Build) (64-bit)### Reproducible demo link### Describe the bugPlease check this.selected has a value of un...
Aravinth-2020
Hey @Vampire-V please follow properly the bug issue template. We can't help you here without a reproducible demo. Can you please share the solution for that? I have a same issue when i select the open layer manager, this issue raises after...
#6039August 6, 2024by Gryphonn3 answers
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome 126 Reproducible demo link https://jsfiddle.net/Gryphonn/ob1pznh6/ Describe the bug How to reproduce the bug?Select any word.Wrap it for style.Keep this word selected.Go to General -> DisplayI want Block. I click Bl...
artf
Yeah, the issue is here: https://github.com/GrapesJS/grapesjs/blob/1e3766f0060e6da502fccf9ba9eb938d5ac860bb/src/style_manager/model/PropertyFactory.ts#L267 Would it make sense to add a new option like initial and make it the default one?
danstarns
> Hi @Gryphonn,> I tried to reproduce your issue but the steps aren't that clear, please rephrase. bandicam.2024-08-08.21-07-14-667.mp4 Thank you for a detailed report 🙏 It looks like we should better handle the default display so you don...
danstarns
Hi @Gryphonn, I tried to reproduce your issue but the steps aren't that clear, please rephrase.
#6038August 6, 2024by dharma-glui1 answer
GrapesJS version[X] I confirm to use the latest version of GrapesJSWhat browser are you using? ChromeReproducible demo link https://jsfiddle.net/6bn43a05/Describe the bug How to reproduce the bug?Add a onclick event for a html tag What is the expected behavior? It should not remove the event after getting loaded in to...
artf
This is intentional for security reasons, but you can disable it with the option config.parser.optionsHtml.allowUnsafeAttr (at your own risk): https://github.com/GrapesJS/grapesjs/blob/819bfcf44bb23d51990eba3b4330e9932df59cde/src/parser/co...
#6032August 3, 2024by aswinkumar8631 answer
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome v126 Reproducible demo link https://grapesjs.com/demo Describe the bug How to reproduce the bug?Open the demoDouble-click on any existing image component to open the asset manager.Drag and drop a video file into the...
artf
Thanks @aswinkumar863 for the report. I'll be happy to merge a PR of the fix.
#6024July 30, 2024by yashvi20261 answer
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome Reproducible demo link https://codepen.io/Harshsne/pen/xxoOyXO Describe the bug https://github.com/GrapesJS/grapesjs/issues/6019 with reference to this, I wanted to hide rte only for some particular type . Example:...
artf
https://github.com/GrapesJS/grapesjs/issues/6019#issuecomment-2267466020 Avoid creating duplicate issues, please.
#6019July 26, 2024by yashvi20263 answers
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome Reproducible demo link https://codepen.io/Harshsne/pen/xxoOyXO Describe the bug How to reproduce the bug?drop text componentedit text component What is the expected behavior? rte shouldnt be displayed What is the cu...
artf
Yeah onActive is now an async method so extending it with extendFnView is not enough, your code is executed before the original one. Anyway, you can achieve the same result with a simple line of CSS
yashvi2026
but i wanted to hide only some particular type eg:text @artf
artf
You can still extend the view without using extendFnView. Reuse the original component type via prototype, as indicated in here.