sungmg/mutter-performance-source
1
0
Fork 0
Code Activity

Return event coordinates for 3BUTTON_PRESS events

Browse source 
The switch() in clutter_event_get_coords() wasn't returning the coordinates
for the 3 button press events, thus those events never reached the actors.
This commit is contained in:
Emmanuele Bassi 2007-08-21 15:48:51 +00:00
parent 59f4df5dfd
commit 00142fdb62
3 changed files with 12 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
clutter/clutter-event.c
View file

@ -154,12 +154,17 @@ clutter_event_get_coords (ClutterEvent *event,
switch (event->type) switch (event->type)
{ {
case CLUTTER_NOTHING:
case CLUTTER_KEY_PRESS: case CLUTTER_KEY_PRESS:
case CLUTTER_KEY_RELEASE: case CLUTTER_KEY_RELEASE:
event_x = event_y = 0; case CLUTTER_STAGE_STATE:
case CLUTTER_DESTROY_NOTIFY:
case CLUTTER_CLIENT_MESSAGE:
case CLUTTER_DELETE:
break; break;
case CLUTTER_BUTTON_PRESS: case CLUTTER_BUTTON_PRESS:
case CLUTTER_2BUTTON_PRESS: case CLUTTER_2BUTTON_PRESS:
case CLUTTER_3BUTTON_PRESS:
case CLUTTER_BUTTON_RELEASE: case CLUTTER_BUTTON_RELEASE:
event_x = event->button.x; event_x = event->button.x;
event_y = event->button.y; event_y = event->button.y;
@ -172,8 +177,6 @@ clutter_event_get_coords (ClutterEvent *event,
event_x = event->scroll.x; event_x = event->scroll.x;
event_y = event->scroll.y; event_y = event->scroll.y;
break; break;
default:
break;
} }
if (x) if (x)
@ -440,6 +443,7 @@ clutter_event_get (void)
if (!(event->flags & CLUTTER_EVENT_PENDING)) if (!(event->flags & CLUTTER_EVENT_PENDING))
{ {
g_queue_remove (context->events_queue, event); g_queue_remove (context->events_queue, event);
return (ClutterEvent *) event; return (ClutterEvent *) event;
} }

8
clutter/clutter-main.c
View file

@ -273,7 +273,7 @@ clutter_do_event (ClutterEvent *event)
x, y, x, y,
CLUTTER_PICK_REACTIVE); CLUTTER_PICK_REACTIVE);
CLUTTER_NOTE (EVENT, "Reactive event recieved at %ix%i - actor: %p", CLUTTER_NOTE (EVENT, "Reactive event received at %i, %i - actor: %p",
x, y, actor); x, y, actor);
if (event->type == CLUTTER_SCROLL) if (event->type == CLUTTER_SCROLL)
@ -303,14 +303,14 @@ clutter_do_event (ClutterEvent *event)
*/ */
while (actor) while (actor)
{ {
if (clutter_actor_is_reactive (actor) if (clutter_actor_is_reactive (actor) ||
|| clutter_actor_get_parent(actor) == NULL /* STAGE */ ) clutter_actor_get_parent (actor) == NULL /* STAGE */ )
{ {
CLUTTER_NOTE (EVENT, "forwarding event to reactive actor"); CLUTTER_NOTE (EVENT, "forwarding event to reactive actor");
clutter_actor_event (actor, event); clutter_actor_event (actor, event);
} }
actor = clutter_actor_get_parent(actor); actor = clutter_actor_get_parent (actor);
} }
} }
break; break;