Code review comment for lp:~widelands-dev/widelands/full_texture_atlas

Revision history for this message
SirVer (sirver) wrote :

> As i understand, merging just this branch is only the preparation for implementing the whole stuff and wouldn't change anything for now. So i am fine with merging.

The full implementation is https://code.launchpad.net/~widelands-dev/widelands/use_image_cache. It implements solution 1. For now it only shows a static text and it freezes Widelands while loading, adding a slideshow or progress report is feasible, but more work.

I have a lot of testing feedback for this, but no code review comments. Could I get a lgtm for the code before merging?

« Back to merge proposal