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.

Sign up for GitHub

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

Jump to bottom

Review for Fritz #17

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Review for Fritz #17

wants to merge 7 commits into from

Commits on Oct 7, 2014

  1. Editor does not know about menu 

    Menu API simplified
Renamed navigator => menuRenderer
Menu does not know about menuRenderer
    @yangit
    yangit committed Oct 7, 2014
    Configuration menu
    Copy the full SHA
    b1eaedb View commit details
    Browse the repository at this point in the history

Commits on Oct 21, 2014

  1. * editor does not care about view or menu 

    * old puddle-syntax removed
* view and menu code refactored
* keycode replaced with keypressJS lib
    @yangit
    yangit committed Oct 21, 2014
    Configuration menu
    Copy the full SHA
    41ea672 View commit details
    Browse the repository at this point in the history

Commits on Nov 7, 2014

  1. This is a big change to client side libraries API. 

    Editor.js is now split among cursor.js, menu-builder.js, forest.js
Event listener is used across libraries to notify each other of events.
See reference.md for better description.
    @yangit
    yangit committed Nov 7, 2014
    Configuration menu
    Copy the full SHA
    ff21837 View commit details
    Browse the repository at this point in the history

Commits on Dec 9, 2014

  1. Fix rendering issue #20

    @yangit
    yangit committed Dec 9, 2014
    Configuration menu
    Copy the full SHA
    8d9a1b5 View commit details
    Browse the repository at this point in the history

  2. Fix #18, puddle-hub too verbose in server console.

    @yangit
    yangit committed Dec 9, 2014
    Configuration menu
    Copy the full SHA
    eaf43f5 View commit details
    Browse the repository at this point in the history

Commits on Dec 11, 2014

  1. Fix #17 Issue 

    Refactor the way cursor is injected into the term and removed from term.
Cursor injection simplified in view.js
    @yangit
    yangit committed Dec 11, 2014
    Configuration menu
    Copy the full SHA
    3de8cca View commit details
    Browse the repository at this point in the history

  2. Fix #19 

    Added line checkout notification across multiple clients.
    @yangit
    yangit committed Dec 11, 2014
    Configuration menu
    Copy the full SHA
    5553be8 View commit details
    Browse the repository at this point in the history