Undo operations by Ctrl+Z

[gohabereg]

[eduardomart]

I'd like to see this too, currently if you delete something, it's gone for good.

[lostpebble]

On the undo-redo note - I've been using immer for a project recently and adding in my own undo / redo functionality was trivial. It is a great library because it offers the ability to use "patches" (https://github.com/mweststrate/immer#patches) which are records of the individual pieces of your JSON tree that you have changed on your object.

The easy of use it brings to editing big JSON objects directly might be useful as a whole to this project, and the patches would be a great addition to allow complete control over undo / redo.

Behind the scenes they look something like this (you get both regular and inverse patches so you can go both ways):

// inverse
[
    {
        "op": "replace",
        "path": ["profile"],
        "value": {"name": "Veria", "age": 5}
    },
    {"op": "remove", "path": ["tags", 3]}
]

// regular
[
    {"op": "replace", "path": ["profile"], "value": {"name": "Noa", "age": 6}},
    {"op": "add", "path": ["tags", 3], "value": "kiddo"}
]

I actually think its the perfect data tool for something like a block editor. I understand not wanting to add dependencies, but this one could be worth looking at and its only 4.2kB gzipped.

[gohabereg]

Hi @lostpebble !

I was working on these features using "patches" too, thanks for pointing out the immer!

This also would be really good for the future collaborative mode

[lostpebble]

This also would be really good for the future collaborative mode

Yep exactly! That's part of the magic - only sending exactly the changes that are being made instead of the entire tree.

This could definitely kill two birds with one stone, undo / redo and collaborative modes.

I would definitely take more of a look at immer as whole too, it really has made my life 10x easier when working on my state library (https://github.com/lostpebble/pullstate) - in fact I made it a core mechanic - as it allows you to mutate your state objects directly and it can figure out when nothing has changed very easily (good for performance checks). Overall a brilliant library!

[gohabereg]

The problem I faced is we need to work with DOM state directly if we don't want to plugin developers care about that. It is a little inconsistent with the idea of working with the data as JS objects. Anyway will look at the immer, thanks!

[lostpebble]

Ah, okay. That is a little disappointing requirement, I was thinking that this was more of a "virtual" DOM as a whole - based completely off of the JSON object.

[gohabereg]

We will think of options how to implement this. I hope we will include it to the next major release

[AlexRMU]

So, 3 years have passed

[bbbford]

Great library guys! Without undo operations, it is hard to envision anyone using this for production apps.

If the DOM could somehow be reactive to the underlying JSON model, then one could fairly easily implement undo/redo and collaborative editing using a CRDT library such as AutoMerge.

Using CDRT, in my opinion, would give this library a huge leg up on the competition, which primarily use Operational Transformation (which from my understanding requires a server to ensure that operations are transformed in the correct order, as opposed to CDRT, which does not care about order of transactions).

[just99]

Hi! Any info about this bug? Can't work without undo-redo =(

[kenyxy]

what is the current progress on the undo/redo feature?
how about the simple idea to keep change history and call Block.render(previousDATA)?

Render
Method removes all Blocks and fills with new passed JSON data

[hiitsgabe]

Hey Guys, any news about this feature?

[hiitsgabe]

On the undo-redo note - I've been using immer for a project recently and adding in my own undo / redo functionality was trivial. It is a great library because it offers the ability to use "patches" (https://github.com/mweststrate/immer#patches) which are records of the individual pieces of your JSON tree that you have changed on your object.

The easy of use it brings to editing big JSON objects directly might be useful as a whole to this project, and the patches would be a great addition to allow complete control over undo / redo.

Behind the scenes they look something like this (you get both regular and inverse patches so you can go both ways):

// inverse
[
    {
        "op": "replace",
        "path": ["profile"],
        "value": {"name": "Veria", "age": 5}
    },
    {"op": "remove", "path": ["tags", 3]}
]

// regular
[
    {"op": "replace", "path": ["profile"], "value": {"name": "Noa", "age": 6}},
    {"op": "add", "path": ["tags", 3], "value": "kiddo"}
]

I actually think its the perfect data tool for something like a block editor. I understand not wanting to add dependencies, but this one could be worth looking at and its only 4.2kB gzipped.

@lostpebble Could you provide an example of your implementation?

[mydearxym]

another possible approach is let editor.js-core maintains an undo/redo stack, once ctrl-Z action triggered, each plugin's renderIfChanged method will be called, just like React's shouldComponentUpdate, then the renderIfChanged method will re-execute the render method.

what you guys think? @gohabereg @eduardomart @lostpebble @bbbford @just99

[kfirba]

@gohabereg @neSpecc Is there any progress regarding this issue?

[rathpanha]

Everyone waiting for this feature ^^

[juanhurtado10]

👋👋👋 We have developed a plugin that can address this feature. We hope this may help :).

https://github.com/kommitters/editorjs-undo

The code is already open and published to NPM ... everyone is welcome to contribute 🎩.

[juanhurtado10]

I forgot to mention ... Redo is also supported ;)

[mydearxym]

👋👋👋 We have developed a plugin that can address this feature. We hope this may help :).

https://github.com/kommitters/editorjs-undo

The code is already open and published to NPM ... everyone is welcome to contribute 🎩.

that's awesome!

but i wonder if the history-stack should only store the diff part between each save data ? otherwise the history-stack will be huge if the article is large.

plus the undo and redo seems lost the caret state ?

[nicolasgaviria]

that's awesome!

but i wonder if the history-stack should only store the diff part between each save data ? otherwise the history-stack will be huge if the article is large.

plus the undo and redo seems lost the caret state ?

@mydearxym The main issue with that approach is that the render method the API provides receives the whole object. We cannot render a specific piece of the blocks.

About the last part, what do you mean with 'caret state'? It would be awesome if you create an issue explaining us the problem.

Thanks!

[gacardinal]

@nicolasgaviria Firstly, thanks for contributing your plugin! It's really cool and I think our organisation will start using it!

However, like @mydearxym suggested, I too think that the history stack should only store the diffs, probably using a command pattern!

The main issue with that approach is that the render method the API provides receives the whole object. We cannot render a specific piece of the blocks.

That's a fair point, but couldn't the plugin compute the whole set of blocks that need to be passed to the editor by replaying the commands according to the user's actions?

I might (please don't hold me accountable 🙃 ) start a fork of either your undo plugin or the editor itself to try implementing it with a command pattern.

I'd argue that it'd be better to have this feature natively in the editor core because, especially to end users but to developers as well, the editor looks like one big <input /> tag, and I think it's like 'expected behavior' to have the Undo / Redo feature because the browsers do implement it on <input /> tags.

I think that the way @nicolasgaviria 's plugin implements it using a DOM MutationObserver is better than using a diffing library simply because of bundle size considerations.

Anyway, those are my two cents, I'll report back if I do end up forking, or creating something else!

[mhmttosun]

Hi,

without Undo/Redo operation, it is really very hard to use editor.

[komaldembla]

👋👋👋 We have developed a plugin that can address this feature. We hope this may help :).

https://github.com/kommitters/editorjs-undo

The code is already open and published to NPM ... everyone is welcome to contribute 🎩.

This is awesome @juanhurtado10!! But, if you have empty blocks in between the data, they are lost while pressing Ctrl + Z

notebook_new_line.mov

[juanhurtado10]

if you have empty blocks in between the data, they are lost while pressing Ctrl + Z

Hey, @komaldembla thanks for reporting.

Just added a new issue in the plugin repo
kommitters/editorjs-undo#53

Could you please drop the following information there?

• Device, Browser, OS:
• Editor.js version:
• Plugins you use with their versions

Best,
Juan!

[komaldembla]

Thanks @juanhurtado10! I've updated the details on this issue itself

[Aarbel]

So is that supported now ?

[juanhurtado10]

@komaldembla @Aarbel

Have you guys tried the following config? I think this allows the undo plugin to recover the empty blocks.

  paragraph: {
    class: Paragraph,
    inlineToolbar: true,
    config: {
      preserveBlank: true,
    },
  },
  ...

let me know if worked for you!

Best,
Juan

Screen.Recording.2021-09-08.at.5.07.51.PM.mov

[komaldembla]

Thank you so much @juanhurtado10. It works like a charm!!

[juanhurtado10]

Hey everyone!!

We just released the v0.3.0 of the Undo/Redo plugin. It:

• Adds support for custom keyboard shortcuts.
• Adds compatibility for the react-editor-js wrapper.

All the info is in the Readme.

Hope it helps 🍻 ... feedback is really appreciated :).

Best
Juan

[evanjmg]

The performance for editorjs-undo is unbearable. I've had to fork the undo plugin to support slices of what's changed. However, I have had no luck in getting what has changed from editorjs (what index or grouping of indices - pages of a document in my case). Further support of the onChange event in plugins would allow one to extend and get the changes correctly. Indices before and after - store them.

[MarioRodriguezS]

@evanjmg Hi there!

We just released a beta version of editorjs-undo (v2.0.0-rc.0) that improve the unexpected behavior. We expect that this helps to solve your issues and if it is not enough or if you find any other issue, please report in the repository of editorjs-undo

More information may be found in the Readme file. Thanks!!

Best,
Mario

[SalahAdDin]

Does this feature replace this plugin?