Code review comment for lp:~widelands-dev/widelands/choose-attack-soldiers
Revision history for this message
| Toni Förster (stonerl) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
73416ba
(Get the code!)
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?