Code review comment for lp:~widelands-dev/widelands/bug-1395322-tool3
Revision history for this message
| kaputtnik (franku) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
73416ba
(Get the code!)
I don't know the reason why Ctrl is used to remove objects at all. In my opinion there should be only one key for removing objects and use this key for all objects. So for all objects:
Click: Place Objects or Increase (meaning 'add something')
Shift + Click: Remove Objects or Decrease (meaning 'remove something')
This should be consistent for all objects, imho. And the Ctrl key should only work for special things (like "Set Value").