← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~widelands-dev/widelands/bug-536489-dropdown into lp:widelands

 

Review: Needs Fixing compile, test

Hello Gun,
thanks for your work.

* the Autoclose when leaving feels reasonbale. OK for me
* Esc-Key is handled as expected.

There is one confusion however: 
The menu has a Selection - via a background color - and a checkmark.
When I use the cursor keys to scroll both, the selection and the
checkmark change. So I assume I made the choice. When I then leave
the list, I find that my selection was not applied (so the checkmark is wrong).

Please improve as follows: do not move the checkmark until the selection
in finally confirmed by clicking or by pressing return.
-- 
https://code.launchpad.net/~widelands-dev/widelands/bug-536489-dropdown/+merge/306303
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-536489-dropdown.


References