Improvements related to editing of features by nprigour · Pull Request #274 · locationtech/udig-platform
some formatting of code
I'm wondering about your format settings. Have you configured codeformatter.xml settings for your workspace (Preferences —> Java —> Code Style —> Code Formatter)?
exception of renaming of runTest ..
I'm fine with it althought it is not convenient naming such methods. It was just a hint to improve codebase while other classes still might have "special" style ;) And, especially in this case, its copied or used as template which makes it even worse.
Besides all the above and before merging the pull request I would like your opinion if it is worthy to
perform also some code refactoring to gather for exapmle all cell editors in a separate package
witihin orgo.locationtech.udig.ui plugin. What do you think?
Good Idea and I'd like to suggest to seperate it from this pull request. We can collaborate on your suggested refacturings (sub-packaging).