Skip to content
This repository was archived by the owner on Jul 24, 2019. It is now read-only.

Added a change event to the editor#50

Open
jordandh wants to merge 1 commit into
mindmup:masterfrom
jordandh:change
Open

Added a change event to the editor#50
jordandh wants to merge 1 commit into
mindmup:masterfrom
jordandh:change

Conversation

@jordandh

Copy link
Copy Markdown
Contributor

Based on what we talked about I went ahead and added the change event to the editor.

@gojko

gojko commented Apr 19, 2013

Copy link
Copy Markdown

I was looking into this as well. I think this needs a bit more love :) I'm travelling today so I won't be able to complete it, but I was thinking about caching the old version and actually comparing it with the current one before triggering change. this should prevent multiple fake triggers and preventing events that don't change content

@jordandh

Copy link
Copy Markdown
Contributor Author

I understand the need to make it as accurate as possible. I am worried about holding onto the old value though. If there are a couple of images in the editor then it could be multiple MBs being stored and compared. That could be slow and consume a decent chunk of memory.

@gojko

gojko commented Apr 20, 2013

Copy link
Copy Markdown

hm... you're right. perhaps a checksum then

@SET001

SET001 commented May 7, 2013

Copy link
Copy Markdown

How to use this to catch changes?
For now i use this

MutationObserver = window.MutationObserver || window.WebKitMutationObserver
observer = new MutationObserver (mutations, observer) ->
    scope.data = editor.html()
observer.observe document,
    subtree: true
    attributes: true

@festaans

Copy link
Copy Markdown

Hi, I'm trying to use bootstrap wysiwyg as a "local" offline editor. I'm currently trying to make use of garlic.js to get the info in the editor persistent but no luck yet. Any ideas?

@mm01

mm01 commented Jan 6, 2014

Copy link
Copy Markdown

how can i bind and use chane event?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants