GrapesJS / grapesjs

Free and Open source Web Builder Framework. Next generation tool for building templates without coding
https://grapesjs.com
BSD 3-Clause "New" or "Revised" License
22.38k stars 4.06k forks source link

[Bug] Creating multiple child element after editing using RTE #1502

Closed mudit-mittal closed 5 years ago

mudit-mittal commented 6 years ago

Hi All,

I am getting below issues which are related to RTE

  1. RTE creating multiple child's as action performed on text block. Ex:- suppose we performed same action (Bold) three times in sequence on same text block then its creating three child's on the same element.
  2. Font action in RTE is only supporting font size values between 1 to 7 so please provide the support for font size values between 1 to 100 in RTE.

image

NP:- I am using latest version of library but issue is still here so kindly review.

Previous bug for reference https://github.com/artf/grapesjs/issues/1495#issue-368617259 Kindly revert on priority. Thanks in Advance !!!

mudit-mittal commented 6 years ago

Hi All,

Please find step for reproducing this scenario

1) Initial state image

2) Perform Bold Action then Italic Action then Underline Action image

3) Again click on Bold Action then Italic Action then Underline Action image

Actual:- Getting three extra element

mudit-mittal commented 6 years ago

Hi All,

rte.add('fontSize', {

this block of code is only supporting font size values between 1 to 7 although is should be generic and should be support all integer values

artf commented 5 years ago

This is just how execCommand works and the built-in RTE is based on it. If you need something more sophisticated you can try grapesjs-plugin-ckeditor or implement any other 3rd-party editor by following this guide: https://grapesjs.com/docs/guides/Replace-Rich-Text-Editor.html

lock[bot] commented 4 years ago

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.