Code review comment for lp:~widelands-dev/widelands/bug-1664052-expedition-shipwindow-crash

Revision history for this message
kaputtnik (franku) wrote :

Ah, ok i see... i meant formerly the black area fits the whole inner window, whereas now there is some space on the left and right between window border and black area. It looks like the black area is in the air somehow.

I have no time (and maybe no experience) for creating such an image i mentioned. But maybe it is possible to have something like shown in the bottom of this mockup: https://bugs.launchpad.net/widelands/+bug/1664052/+attachment/4831701/+files/laderaum_1.png (just compared ship windows: Old style, this branch, and a suggestion)

Otherwise looks good :-)

« Back to merge proposal