dcorking / wave-robot-java-client

Automatically exported from code.google.com/p/wave-robot-java-client
0 stars 0 forks source link

Insertion and deletion of elements #10

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
In the TextViewImpl class the methods "insertElement" and "appendElement" 
implicitly insert a space and additionally a newline. While this seems 
strange to me anyways (it should either generate additional text insertion 
operations or not insert any text at all in my opinion) it is even more 
strange that "deleteElement" implicitly deletes just one character at the 
element's index instead of two (again without any text operations).
This implicit deletion causes another serious problem if some text was 
deleted before the element's index, as the element indexes are not shifted 
back on text deletion (i.e. a call to "shiftElementsUp" from within "delete
(Range)" is missing). Subsequent calls to "deleteElement" would delete the 
wrong characters.

Original issue reported on code.google.com by patrick....@googlemail.com on 21 Jul 2009 at 4:21