JasonWoof Got questions, comments, patches, etc.? Contact Jason Woofenden
fix left/right cursor past non-visible whitespace
[peach-html5-editor.git] / editor.coffee
index f4d0efd..0d6e761 100644 (file)
@@ -19,20 +19,17 @@ TYPE_TEXT = peach_parser.TYPE_TEXT
 TYPE_COMMENT = peach_parser.TYPE_COMMENT
 TYPE_DOCTYPE = peach_parser.TYPE_DOCTYPE
 
-# text nodes don't have getBoundingClientRect(), so wrap it in a span, measure
-# and then put it back
+# text nodes don't have getBoundingClientRect(), so use selection api to find
+# it.
 get_text_bounding_rect = (el) ->
-       span = el.ownerDocument.createElement 'span'
-       el.parentNode.replaceChild span, el
-       span.appendChild el
-       ret = span.getBoundingClientRect()
-       span.parentNode.replaceChild el, span
-       return ret
 get_el_bounds = (el) ->
-       if el.getBoundingClientRect
+       if el.getBoundingClientRect?
                rect = el.getBoundingClientRect()
        else
-               rect = get_text_bounding_rect el
+               # text nodes don't have getBoundingClientRect(), so use range api
+               range = document.createRange()
+               range.selectNodeContents el
+               rect = range.getBoundingClientRect()
        doc = el.ownerDocument.documentElement
        win = el.ownerDocument.defaultView
        y_fix = win.pageYOffset - doc.clientTop
@@ -49,8 +46,7 @@ get_el_bounds = (el) ->
 #
 # implementation: insert a span tag where we want the cursor, and ask the
 # browser where it put that span
-cursor_to_xyh = (n, i) ->
-       span = domify span: style: "height: 1em; border-left: 1px solid black; margin-left: -1px"
+window.cursor_to_xyh = cursor_to_xyh = (n, i) ->
        parent = n.el.parentNode
        els = []
        txts = []
@@ -93,7 +89,7 @@ cursor_to_xyh = (n, i) ->
        # fudge case where bounds are BS because we're on non-significant whitespace
        if i > 0
                first = cursor_to_xyh n, 0
-               if ret.x <= first.x
+               if ret.x <= first.x and ret.y is first.y
                        # no need for a loop here, because recursion
                        ret = cursor_to_xyh n, i - 1
        return ret
@@ -175,17 +171,15 @@ css +=     'height: 1em;'
 css +=     'width: 2px;'
 css +=     'margin-left: -1px;'
 css +=     'margin-right: -1px;'
-css +=     'background: #000;'
-css +=     '-webkit-animation: 1s blink step-end infinite;'
-css +=     'animation: 1s blink step-end infinite;'
+css +=     'background: #444;'
+css +=     '-webkit-animation: blink 1s steps(2, start) infinite;'
+css +=     'animation: blink 1s steps(2, start) infinite;'
 css += '}'
-css += '@-webkit-keyframes "blink" {'
-css +=     'from, to { background: #000; }'
-css +=     '50% { background: transparent; }'
+css += '@-webkit-keyframes blink {'
+css +=     'to { visibility: hidden; }'
 css += '}'
-css += '@keyframes "blink" {'
-css +=     'from, to { background: #000; }'
-css +=     '50% { background: transparent; }'
+css += '@keyframes blink {'
+css +=     'to { visibility: hidden; }'
 css += '}'
 
 # key codes:
@@ -232,7 +226,11 @@ traverse_tree = (tree, state, cb) ->
 # TODO make it so cursor can go places that don't have text but could
 find_next_cursor_position = (tree, n, i) ->
        if n? and n.type is TYPE_TEXT and n.text.length > i
-               return [n, i + 1]
+               orig_xyh = cursor_to_xyh n, i
+               for next_i in [i+1 .. n.text.length] # inclusive is valid (after last char)
+                       next_xyh = cursor_to_xyh n, next_i
+                       if next_xyh.x > orig_xyh.x or next_xyh.y > orig_xyh.y
+                               return [n, next_i]
        found = traverse_tree tree, before: n?, (node, state) ->
                if node.type is TYPE_TEXT and state.before is false
                        state.node = node
@@ -246,6 +244,11 @@ find_next_cursor_position = (tree, n, i) ->
 # TODO make it so cursor can go places that don't have text but could
 find_prev_cursor_position = (tree, n, i) ->
        if n? and n.type is TYPE_TEXT and i > 0
+               orig_xyh = cursor_to_xyh n, i
+               for prev_i in [i-1 .. 0]
+                       prev_xyh = cursor_to_xyh n, prev_i
+                       if prev_xyh.x < orig_xyh.x or prev_xyh.y < orig_xyh.y
+                               return [n, prev_i]
                return [n, i - 1]
        found = traverse_tree tree, before: n?, (node, state) ->
                if node.type is TYPE_TEXT
@@ -270,6 +273,9 @@ find_loc_cursor_position = (tree, loc) ->
                        continue if loc.x > bounds.x + bounds.w
                        continue if loc.y < bounds.y
                        continue if loc.y > bounds.y + bounds.h
+                       if c.children.length
+                               ret = find_loc_cursor_position c.children, loc
+                               return ret if ret?
                        if c.type is TYPE_TEXT
                                # click is within bounding box that contains all text.
                                return [c, 0] if c.text.length is 0
@@ -281,6 +287,12 @@ find_loc_cursor_position = (tree, loc) ->
                                        continue # before first char on first line
                                if loc.y > after.y and loc.x > after.x
                                        continue # after last char on last line
+                               if loc.y < before.y
+                                       console.log "Warning: click in bounding box but above first line"
+                                       continue # above first line (runaround?)
+                               if loc.y > after.y + after.h
+                                       console.log "Warning: click in bounding box but below last line", loc.y, after.y, after.h
+                                       continue # below last line (shouldn't happen?)
                                while after_i - before_i > 1
                                        cur_i = Math.round((before_i + after_i) / 2)
                                        cur = cursor_to_xyh c, cur_i
@@ -295,9 +307,6 @@ find_loc_cursor_position = (tree, loc) ->
                                        return [c, before_i]
                                else
                                        return [c, after_i]
-                       if c.children.length
-                               ret = find_loc_cursor_position c.children, loc
-                               return ret if ret?
        return null
 
 class PeachHTML5Editor
@@ -384,8 +393,18 @@ class PeachHTML5Editor
                                        when KEY_END
                                                return false
                                        when KEY_BACKSPACE
+                                               return false unless @cursor?
+                                               return false unless @cursor[1] > 0
+                                               @cursor[0].text = @cursor[0].text.substr(0, @cursor[1] - 1) + @cursor[0].text.substr(@cursor[1])
+                                               @cursor[0].el.nodeValue = @cursor[0].text
+                                               @move_cursor [@cursor[0], @cursor[1] - 1]
                                                return false
                                        when KEY_DELETE
+                                               return false unless @cursor?
+                                               return false unless @cursor[1] < @cursor[0].text.length
+                                               @cursor[0].text = @cursor[0].text.substr(0, @cursor[1]) + @cursor[0].text.substr(@cursor[1] + 1)
+                                               @cursor[0].el.nodeValue = @cursor[0].text
+                                               @move_cursor [@cursor[0], @cursor[1]]
                                                return false
                                        when KEY_ENTER
                                                return false
@@ -443,7 +462,6 @@ class PeachHTML5Editor
                @clear_dom()
                instantiate_tree @tree, @idoc.body
        move_cursor: (cursor) ->
-               return if @cursor? and cursor? and @cursor[0] is cursor[0] and @cursor[1] is cursor[1]
                @cursor = cursor
                # replace cursor, to reset blink animation
                if @cursor_visible