← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~widelands-dev/widelands/savegame-menu into lp:widelands

 

Had a quick look and it LGTM :-)

Don't know if this related to your changes: 
- Fullscreen toggling does not work for the save game menu, as well for Editor Load/Save map menu. But for me this is ok.
- I have some incompatible maps showing an Error message. Hovering with the mouse over the errormessage shows the tooltip "The Map that this map is based on" of the map.
- The tooltip is only shown when hovering exactly over the first line. 
- In trunk there is also a tooltip for the Win Condition, which is missing here

General in regard to the tooltips when hovering "Map name" and "Win condition": I don't know if they are use full in the Load game menu, because they are telling the same as the texts already says. So they do not help in any way, but causes additional work for the translators, imho.
-- 
https://code.launchpad.net/~widelands-dev/widelands/savegame-menu/+merge/322924
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/savegame-menu.


Follow ups

References