GNOME Bugzilla – Bug 594074
Overview still interactive during animation
Last modified: 2009-09-14 19:34:52 UTC
Reproduce: 1. Open Activities 2. Dubble click on any item in the Places dialog (example: Documents) This will open 2 windows of Documents however the effect should either be dubble click to open a window or single click to open a window. Only 1 window should be opened however. Signle Click and 1 window open would be the best to follow the rest of the settings in the activities menu.
Maybe have Main.overview.hide just raise the transparent background over everything until the animation is complete.
We have a variety of bugs like this; another one is that the mouseover handler for WindowClone which does raise_top still runs during the leave animation which screws up the stacking order.
Created attachment 143113 [details] [review] [Overview] Make content area eat mouse events during animations There are ton of different kinds of mouse even handlers in the overview; WindowClone has several mouse-enter/leave handlers, we still have a variety of classes not ported to ButtonBox and so incorrectly handling double-click, etc. Since we at present don't have anything in the overview area for which it makes sense to interact with during the animation, create a transparent event-eating box which we raise to the top during the animation.
Comment on attachment 143113 [details] [review] [Overview] Make content area eat mouse events during animations >+ // Similar to the cover pane but used for dialogs ("panes"); see the comments > // in addPane below. > this._transparentBackground = new Clutter.Rectangle({ opacity: 0, > reactive: true }); Is there any reason we need coverPane and can't just use transparentBackground for both? >+ this._dash.actor.set_position(0, contentY); >+ this._dash.actor.set_size(displayGridColumnWidth, screenHeight - contentY); can change the "screenHeight - contentY" to "contentHeight"
(In reply to comment #4) > (From update of attachment 143113 [details] [review]) > >+ // Similar to the cover pane but used for dialogs ("panes"); see the comments > > // in addPane below. > > this._transparentBackground = new Clutter.Rectangle({ opacity: 0, > > reactive: true }); > > Is there any reason we need coverPane and can't just use transparentBackground > for both? I considered that but they feel like two separate things to me; the transparentBackground is more about implementing modal dialog-like behavior. > >+ this._dash.actor.set_position(0, contentY); > >+ this._dash.actor.set_size(displayGridColumnWidth, screenHeight - contentY); > > can change the "screenHeight - contentY" to "contentHeight" Fixed.