widelands-dev team mailing list archive

Thread
Date

Re: [Merge] lp:~widelands-dev/widelands/choose-attack-soldiers into lp:widelands

To: mp+367041@xxxxxxxxxxxxxxxxxx
From: Toni Förster <toni.foerster@xxxxxxxxx>
Date: Fri, 10 May 2019 17:49:07 -0000
In-reply-to: <155723180254.14464.8140195130929090073.launchpad@ackee.canonical.com>
Reply-to: mp+367041@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Sorry to bother you once more :(

Could you make the slider 210px wide, please? Then we would have a maximum of 10 soldiers per row, which looks even better. :)

Regarding the tooltip. Putting them in the text area where the soldier's info are shown is too intrusive, IMHO. Can text areas also hold tooltips?

On the other hand these tooltips are quite massive. Why don't we add them to the controls-list in the encyclopedia?
-- 
https://code.launchpad.net/~widelands-dev/widelands/choose-attack-soldiers/+merge/367041
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/choose-attack-soldiers into lp:widelands.

References

[Merge] lp:~widelands-dev/widelands/choose-attack-soldiers into lp:widelands
From: Benedikt Straub, 2019-05-07