diff options
| author | Frank Reininghaus <[email protected]> | 2011-12-04 13:37:51 +0100 |
|---|---|---|
| committer | Frank Reininghaus <[email protected]> | 2011-12-04 13:54:05 +0100 |
| commit | 82b20ff58e9448868cf23783dcaf1fe304707791 (patch) | |
| tree | 6f6669043c1ad3aa4f115f4e25c66c868a883ded /src/views | |
| parent | e2e5b0e45a0c6e2a943a56e5f4751b70c66104e7 (diff) | |
Clear the selection in MousePressEvent() rather than MouseReleaseEvent()
This fixes the problem that clicking an unselected item in order to drag
it would result in dragging all previously selected items as well. With
this commit, previously selected items are unselected when a new item is
clicked.
The reason why clearing the selection was moved to MouseReleaseEvent()
in commit b583dd6d4d3a03e3af2ec8d370132b84935ff871 was that clicking one
of several selected items should not result in unselecting the other
items (to make sure that dragging multiple items is possible). However,
this can also be assured by just checking in MousePressEvent() if the
clicked item has been selected already and not clearing the previous
selection in that case. This applies equally to the case that a context
menu is requested when several items are selected.
Diffstat (limited to 'src/views')
0 files changed, 0 insertions, 0 deletions