caret-not-following-chain -- Text caret does not go to the prev/next chained frame

[deyan]

• Make the text caret go to next/previous frame (and page if needed).
• The frame containing the caret should be focused.
• If you select from right to left, the caret should be at the beginning of the selection.

[deyan]

Batch update from file active_tickets.csv

[diana]

In order to provide the wanted functionality the following changes were made:

• In org.sophie2.base.model.text.elements package make a new class CaretInfo - a class representing the information about the caret. Has a caret index and area index field. Used when navigating in the text. This class has a default constructor - public CaretInfo(int areaIndex, int caretIndex) and getters for the area index (public int getAreaIndex()) and for the caret index (public int getCaretIndex()).
• In TextModelLogic create an inner enum: public enum EventIds - The operations for the TextModelLogic. It has one field: SELECT_VIEW with @EventParams({Integer.class}).

The event parameter is the area index in case of chaining which sholud be selected after navigating, deleting ot inserting text.

• In TextModelLogic class make the getPos function return Object of type CaretInfo. The area index of this object is used to fire another event to select the proper view.
• In TextModelLogic class make setSelection function fire another event to select the wanted view.
• In HotTextLogic class add ON_SELECT_VIEW - handles the cases when the view is changed (when navigating/editing in chains).

[meddle]

• There is nothing about this isAutoChainMode parameter when creating the text action.
• Why are you changing the fuzz factor
• The design is not full for me but I am passing it to not waste your time, I expect detailed design notes for everything changed in the implementation section.
• Name unit test appropriately - {name}Test
  • The annotations @before and @after @test are important.
  • When you change something in a base test class, I must give explanation why.
• You CAN NOT score analysis without review comments especially with the maximum score.

3p

[diana]

• In HeadTextFrameH class : createTextFrameAction function add another parameter - isAutoChainMode (true if the created frame will be in AUTO_CHAIN mode).
• In HotLayout class change the LAZY_FACTOR to 1 in order to force the layout to layout the text one frame before and after the current one (needed when deleting/ inserting in the beginning/ end of a chained frame).
• In TextUtils : findPrevChar check if the given pos is the end pos and advance backwards if so(needed when the TextFontHaloMenu gets its styles from the text).
• In TextUtils : findNextChar if the given index is the end of the text then the start index should be returned.

In TextFrameLogic class every assert ImmTextUtils.isIndexInText(pos, text) line is not correct because thete is another case which handles the end position of the text (checks if the returned unit == null), so every line of this type is replaced with : assert ImmTextUtils.isIndexInText(pos, text)

pos == text.getEnd();

• In TextModelLogic : setSelection function remove the ImmTextInterval parameter andd add another two - one for the mark and one for the caret position (this is needed because the constructor of the ImmTextInterval reverts the first and second point if they are in reverse order but the mark and caret should not be reversed).

And as for the analysis it's the BEST I'VE EVER READ !!!! :) and I'm speechless....

[pap]

• Commited in [9156].
• Improper JavaDoc of the test :) And you stated boyan as the author...
• Don't swallow exceptions. This applies to tests, too. Actually in tests it is best to just put a throws statement.
• Isn't "textModel.getCaret() - textModel.getMark() <= 0" the same as "textModel.getCaret() <= textModel.getMark()" (TextModelLogic)?
• Some very nice paragraph related changes in TextModelLogic, that weren't mentioned anywhere. :)
• Hah, you just swapped two segments of code.
• Oh and I really enjoy the DOC_HOME and DOC_END navigation.

[meddle]

Closing all the tickets before M Y1