brl/gnome-shell
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Select an item when information button is clicked, launch on single click

Browse Source 
Clicking the information button for an item selects it (i.e. highlights it)
and shows details about the item.
Clicking the rest of the item area launches it.
Item does not become draggable if the dragging is started over the information
icon (i.e. if the user presses the information icon, but releases elsewhere).

Make sure we emit "activated" signal and close the overlay when an item from
one of the results displays is launched.
This commit is contained in:
Marina Zhurakhinskaya 2009-06-18 19:53:21 -04:00
parent 97b9ccbff7
commit c41902c188
2 changed files with 25 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
js/ui/genericDisplay.js
View File

@ -63,15 +63,7 @@ GenericDisplayItem.prototype = {
width: availableWidth,
height: ITEM_DISPLAY_HEIGHT });
this.actor._delegate = this;
this.actor.connect('button-press-event',
Lang.bind(this,
function(actor, e) {
let clickCount = Shell.get_button_event_click_count(e);
if (clickCount == 1)
this.select();
else if (clickCount == 2)
this.activate();
}));
this.actor.connect('button-release-event', Lang.bind(this, this.activate));
let draggable = DND.makeDraggable(this.actor);
draggable.connect('drag-begin', Lang.bind(this, this._onDragBegin));
@ -90,7 +82,16 @@ GenericDisplayItem.prototype = {
});
let global = Shell.Global.get();
this._informationButton.set_from_file(global.imagedir + "info.svg");
this._informationButton.connect('button-release-event', this.select);
// Connecting to the button-press-event for the information button ensures that the actor,
// which is a draggable actor, does not get the button-press-event and doesn't initiate
// the dragging, which then prevents us from getting the button-release-event for the button.
this._informationButton.connect('button-press-event',
Lang.bind(this,
function() {
return true;
}));
this._informationButton.connect('button-release-event', Lang.bind(this, this.select));
this._informationButton.hide();
this.actor.add_actor(this._informationButton);
this._informationButton.lower_bottom();
@ -151,14 +152,16 @@ GenericDisplayItem.prototype = {
this._bg.background_color = color;
},
// Activates the item, as though it was launched
// Activates the item by launching it
activate: function() {
this.emit('activate');
return true;
},
// Selects the item, as though it was clicked
// Selects the item by highlighting it and displaying it details
select: function() {
this.emit('select');
return true;
},
/*

6
js/ui/overlay.js
View File

@ -393,6 +393,12 @@ Dash.prototype = {
this._docDisplay.connect('activated', function(docDisplay) {
me.emit('activated');
});
this._resultsAppsSection.display.connect('activated', function(resultsAppsDisplay) {
me.emit('activated');
});
this._resultsDocsSection.display.connect('activated', function(resultsDocsDisplay) {
me.emit('activated');
});
this._appDisplay.connect('selected', function(appDisplay) {
// We allow clicking on any item to select it, so if an
// item in the app display is selected, we need to make sure that