Thank you Emily for your feedback
I'd like to add some lines about the motivation.
In the last few month I was struggeling fixing bugs and due to the duplications of code it requires to maintain both editor implementations. This increases the effort to maintain the (probably unused) code. Since the unittests covers only the one which is configured (MapEditorPartWithPalette), the MapEdior is quite un-tested - untested code doesn't work :P
--
Frank
I am not using it and ok with having it removed.
Emily
On 2021-10-28 12:29 a.m., Frank Gasdorf wrote:
> Hello everybody & sorry for cross-posting!
>
> Current State
> ---------------------------
> We have to MapEditorPart Implementations in the codebase:
> * MapEditor
> * MapEditorWithPalette
>
> Later is the defaut in uDig for years, while the first still exists.
>
> Request for Change
> ---------------------------
> Since MapEditor is still in the codebase I recommend to retire and
> delete it.
> To get a picture I kindly ask you, whether anybody is still using it in
> projects (using plugin.xml and configure mapEditorOld rather than
> mapEditor)?
>
> Thanks a ton!
>
> --
> Frank
>
>
> _______________________________________________
> udig-dev mailing list
> udig-dev@xxxxxxxxxxx
> To unsubscribe from this list, visit https://www.eclipse.org/mailman/listinfo/udig-dev
>
_______________________________________________
udig-dev mailing list
udig-dev@xxxxxxxxxxx
To unsubscribe from this list, visit https://www.eclipse.org/mailman/listinfo/udig-dev