Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Terminal.diff and apply #14

Merged
merged 12 commits into from
Apr 30, 2013
Merged

Conversation

jedi4ever
Copy link
Contributor

Ok, I know how hard it is to receive these kind of extended pullrequest, so let me begin by apologising.

The pull request is meant to be the basis of an extended diff and apply (including meta info like cursor , mode, led changes etc...) .

It should not interfere right now with any of the existing code. With it I'm now able to do:

 var diff = term1.diff(term2);
 term1.apply(diff); //puts term1 in the same state as term2

By writing this patch, I've also added the necessary primitives/commands that a listener of events can use. The next step(s):

  1. I'd like to take is indeed write the event mechanism (and replace the browserify to make the events also possible for use in the browser.
  2. Write the Webrenderer that can receive this events or changes through a patch

Please consider merging this patch, I'll be happy to talk/discuss/document this to you.
Or talk about it online (drop me mail at github AT jedi DOT be )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants