Hi Saros-Team,

 

as part of the current "Softwareprozesse" lecture here we (Alexander Kurth
and I) were tasked to work on an OSS software bug or feature for a few
weeks. We have chosen to tackle a non-critical feature in Saros/I, enabling
support for displaying multiple carets in IntelliJ IDEA (Issue #116).

 

We dived into the code and I would like to get some feedback regarding our
solution approach before starting the implementation, especially since there
seem to be multiple possibilities, such as:

 

1.       Display the caret's position upon text entry by the user through
the TextEditActivity.

This would be an easy approach, because it requires the smallest changes.
But the big downside is that the caret will only be updated after a user
edited some text, not if it is only being moved around.



2.       Display the caret's position upon moving by the user through the
TextSelectionActivity.

We can read the carets position with the CaretListener, but there seems to
be no specific activity to transmit the updated caret. We could use the
TextSelectionActivity with a length parameter of 0 to signal a moved caret
and handle it in LocalEditorManipulator.selectText?

 

There will probably be even other approaches that we didn't consider yet,
and we would value your input on this.

 

In another matter, we couldn't find any test cases related to the editor
classes. Could you maybe kindly point us in the right direction? Thank you
in advance.

 

Best regards,

Jacob Schramm




------------------------------------------------------------------------------
_______________________________________________
DPP-Devel mailing list
DPP-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dpp-devel

Reply via email to