Code review comment for lp:~widelands-dev/widelands/choose-attack-soldiers

Revision history for this message
Benedikt Straub (nordfriese) wrote :

Implemented the changes as you suggested :)

> The tooltips for the lists hide the line were the soldier’s strength is shown.

I´m not sure how to solve this best: putting the textarea with the info on top wouldn´t look good IMHO, and the tooltip is long enough and shouldn´t contain this as well.
Perhaps I could swap the contents of the textarea and the tooltip, so the explanation what happens when clicking is displayed in the bottom line, and the info of the soldier under the mouse is shown as a tooltip that contains nothing else. What do you think?

« Back to merge proposal