The changes look OK to me. I don't want to mess with the "invalid" icon tag though, because it was added by the Debian folks so that we could have Widelands in the Debian software centers. I have spent many days on this dratted file and I am afraid to touch it again while it's working.
Can you please create a merge request against trunk here, so that we can see a diff?
Thanks for taking this on! :)
The changes look OK to me. I don't want to mess with the "invalid" icon tag though, because it was added by the Debian folks so that we could have Widelands in the Debian software centers. I have spent many days on this dratted file and I am afraid to touch it again while it's working.
Can you please create a merge request against trunk here, so that we can see a diff?