Review: Approve diff

Not tested but code is looking good.
One question: Why checking for both the localized and "plain" name variant when 
saving? In English they should resolve to the same name anyway (right?) and I 
don't see where the plain version can appear otherwise.
-- 
https://code.launchpad.net/~widelands-dev/widelands/bug-1636586-editor-player-menu-i18n/+merge/345464
Your team Widelands Developers is subscribed to branch 
lp:~widelands-dev/widelands/bug-1636586-editor-player-menu-i18n.

_______________________________________________
Mailing list: https://launchpad.net/~widelands-dev
Post to     : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp

Reply via email to