Code review comment for lp:~evfool/synaptic/ancientfixes

Revision history for this message
Michael Vogt (mvo) wrote :

While doing the review I noticed another small issue:

The fix for bug #755548 is not ideal as the code supports multiple selection and its actually a quite
useful feature IMO. It does not work with the "space" key very well, so we would either need to find
a way to make "space" on multiple selections clear the selection and do nothing else (so that the second
space triggers the action). Or make the space trigger the best possible action on all selection.

« Back to merge proposal