← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~widelands-dev/widelands/bug-1653460-panel-init-width into lp:widelands

 

Replied to your question. Thanks for the review!

Diff comments:

> 
> === modified file 'src/ui_fsmenu/loadgame.cc'
> --- src/ui_fsmenu/loadgame.cc	2017-02-26 11:57:15 +0000
> +++ src/ui_fsmenu/loadgame.cc	2017-03-02 08:46:01 +0000
> @@ -188,6 +188,7 @@
>  		ta_mapname_.set_tooltip(_("The map that this game is based on"));
>  		delete_.set_tooltip(_("Delete this game"));
>  	}
> +	set_thinks(false);

There are no background actions for this panel to watch for, so we can save some computing power here.

>  	minimap_icon_.set_visible(false);
>  
>  	back_.sigclicked.connect(boost::bind(&FullscreenMenuLoadGame::clicked_back, boost::ref(*this)));


-- 
https://code.launchpad.net/~widelands-dev/widelands/bug-1653460-panel-init-width/+merge/318358
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-1653460-panel-init-width.


References