Refactor TextTreeItem.java

[arturhg]

Summary

This pull request introduces several refinements and optimizations to the TextTreeItem class in the application. The changes simplify event handling and improve the efficiency of property listeners and text wrapping logic.

Changes

• Removal of Unused Import: Removed the unused Log import to clean up the codebase.
• Code Streamlining:
  • Condensed multiple instances of verbose change listeners into single-line lambda expressions. This change not only cleans up the code but also enhances readability.
  • Replaced multiple condition checks for keyboard events with a streamlined approach using Stream.
• Optimization of Text Wrapping:
  • Replaced string concatenation in the text wrapping logic with StringBuilder to enhance performance and memory management.
  • Adjusted the text wrapping logic to handle new lines more efficiently.
• Refactoring Event Handling:
  • Simplified keyboard event handling to prevent default actions more concisely.
  • Improved focus handling to check equality more robustly using Objects.equals().

Impact

The modifications are expected to enhance the performance of the application by reducing overhead associated with string operations and event handling. The refactoring brings more maintainable and cleaner code, reducing potential bugs and improving the ease of future enhancements. There should be no negative side effects on existing functionalities as the changes focus primarily on internal improvements.

[ClementGre]

I like the idea of code streamlining, but here, I would prefer to have three lines listeners rather than one line listeners. The readability would be decreased by inlining everything. You can revert this part of the code and I’ll merge this.

[arturhg]

I like the idea of code streamlining, but here, I would prefer to have three lines listeners rather than one line listeners. The readability would be decreased by inlining everything. You can revert this part of the code and I’ll merge this.

Hi @ClementGre,

Thank you for your review.

I have reverted the code for the listeners as requested. I look forward to your re-review and the potential merge of this PR.