diff options
author | Shawn Rutledge <shawn.rutledge@qt.io> | 2018-08-31 13:42:22 +0200 |
---|---|---|
committer | Shawn Rutledge <shawn.rutledge@qt.io> | 2018-08-31 13:59:30 +0000 |
commit | 35caa44992c32bd51e55b7b039df8fad7c945bc6 (patch) | |
tree | ca8c860c4d1cc55f17e83c476cdba75a86ee963c /src/quick/items/qquickevents_p_p.h | |
parent | bdb18e33e8b8c7e62514849378107a47a3667431 (diff) |
Rename QQEventPoint::GrabState to GrabTransition
This enum represents a transient state transition, and only sometimes
corresponds to the current grab state of an event point. For example
after exclusive grab has been canceled, the current state is that
there is no exclusive grab: it doesn't make sense to remember that the
way it got there was by cancellation. There was an idea to add a
grabState property, but not all values would be eligible. An
EventPoint can be exclusively grabbed by one item or handler at a
time, and by multiple passive grabbers at the same time, so even a
Q_FLAG would not fully express all possible states. Besides, there is
already an exclusiveGrabber property, and we could add a
passiveGrabbers list property if we had a real need. So adding a
grabState property seems unlikely, and therefore is not a good enough
reason to keep this enum named as GrabState.
Change-Id: Ie37742b4bd431a7e51910d79a7223fba9a6bd848
Reviewed-by: Jan Arve Sæther <jan-arve.saether@qt.io>
Diffstat (limited to 'src/quick/items/qquickevents_p_p.h')
-rw-r--r-- | src/quick/items/qquickevents_p_p.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/quick/items/qquickevents_p_p.h b/src/quick/items/qquickevents_p_p.h index 4dec23d3ae..3623dcdf3a 100644 --- a/src/quick/items/qquickevents_p_p.h +++ b/src/quick/items/qquickevents_p_p.h @@ -281,7 +281,7 @@ public: Q_DECLARE_FLAGS(States, State) Q_FLAG(States) - enum GrabState { + enum GrabTransition { GrabPassive = 0x01, UngrabPassive = 0x02, CancelGrabPassive = 0x03, @@ -290,7 +290,7 @@ public: UngrabExclusive = 0x20, CancelGrabExclusive = 0x30, }; - Q_ENUM(GrabState) + Q_ENUM(GrabTransition) QQuickEventPoint(QQuickPointerEvent *parent); |