Fixes a number of bugs related to unnoting part of a note:
state() was sometimes returning the wrong state. Which meant an Exception and that nothing would happen.
A <SPAN> that we insert for caret positioning caused styling issues. Changed toa zero width char. Not ideal, and we should have a look at that later. Should be good enough for now.
In unnotePartOfNote() the arrays we zip together would sometimes be out of sync. Changed them to use the same source of data for that.
The fix for the previous point also sorted out a caret positioning issue where the caret would be placed at an incorrect position after unnoting a part of a note.
Previously it wasn't possible to select the entire contents of a note and unnote it.
When selecting from 3 paragraphs and then unnoting part of paragraph 2, then the leftmost part of paragraph 3 did not get a note--start tag.
hmgibson23
commented
10 years ago
Fixes a number of bugs related to unnoting part of a note:
state()was sometimes returning the wrong state. Which meant an Exception and that nothing would happen.<SPAN>that we insert for caret positioning caused styling issues. Changed toa zero width char. Not ideal, and we should have a look at that later. Should be good enough for now.unnotePartOfNote()the arrays we zip together would sometimes be out of sync. Changed them to use the same source of data for that.note--starttag.