Use clutter_event_get_* instead of ShellGlobal

Before Clutter gained accessors for event information, we had
shell_global_ functions.  Now that Clutter has them, use them and
delete the ShellGlobal code.

http://bugzilla.gnome.org/show_bug.cgi?id=594561
This commit is contained in:
Colin Walters 2009-09-08 16:58:57 -04:00
parent 5880b3b0ed
commit 37ee16b34d
9 changed files with 10 additions and 56 deletions

View File

@ -108,16 +108,15 @@ iconButton.prototype = {
this.actor.set_opacity(0); this.actor.set_opacity(0);
parent.connect("enter-event", Lang.bind(this, function(actor, event) { parent.connect("enter-event", Lang.bind(this, function(actor, event) {
this._shouldHide = false; this._shouldHide = false;
// Nothing to do if the cursor has come back from a child of the parent actor // Nothing to do if the cursor has come back from a child of the parent actor
if (actor.get_children().indexOf(Shell.get_event_related(event)) != -1) if (actor.get_children().indexOf(event.get_related()) != -1)
return; return;
this._fadeIn(); this._fadeIn();
})); }));
parent.connect("leave-event", Lang.bind(this, function(actor, event) { parent.connect("leave-event", Lang.bind(this, function(actor, event) {
// Nothing to do if the cursor has merely entered a child of the parent actor // Nothing to do if the cursor has merely entered a child of the parent actor
if (actor.get_children().indexOf(Shell.get_event_related(event)) != -1) if (actor.get_children().indexOf(event.get_related()) != -1)
return; return;
// Remember that we should not be visible to hide the button if forceShow is unset // Remember that we should not be visible to hide the button if forceShow is unset

View File

@ -641,7 +641,7 @@ Dash.prototype = {
})); }));
this._searchEntry.entry.connect('key-press-event', Lang.bind(this, function (se, e) { this._searchEntry.entry.connect('key-press-event', Lang.bind(this, function (se, e) {
let text = this._searchEntry.getText(); let text = this._searchEntry.getText();
let symbol = Shell.get_event_key_symbol(e); let symbol = e.get_key_symbol();
if (symbol == Clutter.Escape) { if (symbol == Clutter.Escape) {
// Escape will keep clearing things back to the desktop. // Escape will keep clearing things back to the desktop.
// If we are showing a particular section of search, go back to all sections. // If we are showing a particular section of search, go back to all sections.

View File

@ -418,7 +418,7 @@ LookingGlass.prototype = {
return true; return true;
})); }));
this._entry.connect('key-press-event', Lang.bind(this, function(o, e) { this._entry.connect('key-press-event', Lang.bind(this, function(o, e) {
let symbol = Shell.get_event_key_symbol(e); let symbol = e.get_key_symbol();
if (symbol == Clutter.Escape) { if (symbol == Clutter.Escape) {
this.close(); this.close();
return true; return true;

View File

@ -155,7 +155,7 @@ function _globalKeyPressHandler(actor, event) {
let type = event.type(); let type = event.type();
if (type == Clutter.EventType.KEY_PRESS) { if (type == Clutter.EventType.KEY_PRESS) {
let symbol = Shell.get_event_key_symbol (event); let symbol = event.get_key_symbol();
if (symbol == Clutter.Print) { if (symbol == Clutter.Print) {
// We want to be able to take screenshots of the shell at all times // We want to be able to take screenshots of the shell at all times
let gconf = Shell.GConf.get_default(); let gconf = Shell.GConf.get_default();
@ -169,7 +169,7 @@ function _globalKeyPressHandler(actor, event) {
return true; return true;
} }
} else if (type == Clutter.EventType.KEY_RELEASE) { } else if (type == Clutter.EventType.KEY_RELEASE) {
let symbol = Shell.get_event_key_symbol (event); let symbol = event.get_key_symbol();
if (symbol == Clutter.Super_L || symbol == Clutter.Super_R) { if (symbol == Clutter.Super_L || symbol == Clutter.Super_R) {
// The super key is the default for triggering the overview, and should // The super key is the default for triggering the overview, and should
// get us out of the overview when we are already in it. // get us out of the overview when we are already in it.

View File

@ -472,14 +472,14 @@ Panel.prototype = {
}, },
_onHotCornerLeft : function(actor, event) { _onHotCornerLeft : function(actor, event) {
if (Shell.get_event_related(event) != this._hotCornerEnvirons) { if (event.get_related() != this._hotCornerEnvirons) {
this._hotCornerEntered = false; this._hotCornerEntered = false;
} }
return false; return false;
}, },
_onHotCornerEnvironsLeft : function(actor, event) { _onHotCornerEnvironsLeft : function(actor, event) {
if (Shell.get_event_related(event) != this._hotCorner) { if (event.get_related() != this._hotCorner) {
this._hotCornerEntered = false; this._hotCornerEntered = false;
} }
return false; return false;

View File

@ -132,7 +132,7 @@ RunDialog.prototype = {
})); }));
this._entry.connect('key-press-event', Lang.bind(this, function(o, e) { this._entry.connect('key-press-event', Lang.bind(this, function(o, e) {
let symbol = Shell.get_event_key_symbol(e); let symbol = e.get_key_symbol();
if (symbol == Clutter.Escape) { if (symbol == Clutter.Escape) {
this.close(); this.close();
return true; return true;

View File

@ -296,7 +296,7 @@ WidgetBox.prototype = {
this.state == Widget.STATE_POPPING_OUT)) { this.state == Widget.STATE_POPPING_OUT)) {
// If moving into another actor within this._hbox, let the // If moving into another actor within this._hbox, let the
// event be propagated // event be propagated
let into = Shell.get_event_related(event); let into = event.get_related();
while (into) { while (into) {
if (into == this._hbox) if (into == this._hbox)
return false; return false;

View File

@ -295,47 +295,6 @@ shell_clutter_texture_set_from_pixbuf (ClutterTexture *texture,
0, NULL); 0, NULL);
} }
/**
* shell_get_event_key_symbol:
*
* Return value: Clutter key value for the key press and release events,
* as specified in clutter-keysyms.h
*/
guint16
shell_get_event_key_symbol(ClutterEvent *event)
{
g_return_val_if_fail(event->type == CLUTTER_KEY_PRESS ||
event->type == CLUTTER_KEY_RELEASE, 0);
return event->key.keyval;
}
/**
* shell_get_button_event_click_count:
*
* Return value: click count for button press and release events
*/
guint16
shell_get_button_event_click_count(ClutterEvent *event)
{
g_return_val_if_fail(event->type == CLUTTER_BUTTON_PRESS ||
event->type == CLUTTER_BUTTON_RELEASE, 0);
return event->button.click_count;
}
/**
* shell_get_event_related:
*
* Return value: (transfer none): related actor
*/
ClutterActor *
shell_get_event_related (ClutterEvent *event)
{
g_return_val_if_fail (event->type == CLUTTER_ENTER ||
event->type == CLUTTER_LEAVE, NULL);
return event->crossing.related;
}
/** /**
* shell_global_get: * shell_global_get:
* *

View File

@ -36,10 +36,6 @@ GType shell_global_get_type (void) G_GNUC_CONST;
gboolean shell_clutter_texture_set_from_pixbuf (ClutterTexture *texture, gboolean shell_clutter_texture_set_from_pixbuf (ClutterTexture *texture,
GdkPixbuf *pixbuf); GdkPixbuf *pixbuf);
guint16 shell_get_event_key_symbol(ClutterEvent *event);
guint16 shell_get_button_event_click_count(ClutterEvent *event);
ClutterActor *shell_get_event_related(ClutterEvent *event); ClutterActor *shell_get_event_related(ClutterEvent *event);
ShellGlobal *shell_global_get (void); ShellGlobal *shell_global_get (void);