anilgulecha/trelby — comment on commit 14d84bde89f64a054dc1b4daf5b1a217b706907b
File:  line 0
By anilgulecha:

1) Ack.. I dd not notice the change in the function name.. will implement this in a different way -- either a new function, or change the name/comment.

2) I change to reformat all for a specific reason.. a bug that does not popup with single lines. Selection is tracked by Trelby via line/column numbers. So you have a situation when 10 action lines, when selected and change to dialogue, would convert to x (>>10) lines with mark set only to say half of the old text. This is the reason for clearing the mark.

For reformatting in the above scenario, if we went with the previous method of rewrap elements, we would also need to keep track and figure out what x exactly is, so you rewrap all of the x lines/elements (and not 10 lines/elements).

A simple way around this is reformat all.. but only when multiple lines are selected. I'll figure out a better implementation, so reformat all is not called on simple change type -- when no text selection has been made.

(I hope I was clear with the above explanation.. been a long day)