Code review comment for lp:~widelands-dev/widelands/bug-1560454-mapdir

Revision history for this message
kaputtnik (franku) wrote :

Ahhh... now i got it. I've always tested the map name edit box (as the commit message says, even after you've changed it) but you meant the directory name edit box... This is a really good enhancement :-)

Showing the relative directory is IMHO ok. The position of this could be discussed... but i think we should discuss this if we have time to make a menu overhaul. I want to make some mockups with my suggestions if i am ready with the website code.

The "f" button toggles fullscreen without "CTRL" here. Just like it is explained in the help ;)

review: Approve (testing)

« Back to merge proposal