Code review comment for lp:~widelands-dev/widelands/add-xdg-support

Revision history for this message
GunChleoc (gunchleoc) wrote :

I have done more testing and it all works as expected.

Since it falls back to the old scheme when a .widelands directory is present (which is more convenient for me when developing), I think we could make this the default now.

I'd also still like to see the refactoring done to get rid of the code duplication.

« Back to merge proposal