Created attachment 147701 [details] [review]
[overview] Integer align workspace and pane positioning

Fix works fine for me, solves the "compressed" display in the applications and recent document list

Comment on attachment 147701 [details] [review]
[overview] Integer align workspace and pane positioning

>+            this._activeDisplayPane.actor.width = Math.floor(displayGridColumnWidth * 2);

but, seen in the context part of the diff:

>         this._dash.actor.set_size(displayGridColumnWidth, contentHeight);

which will presumably cause problems elsewhere? Should we just make sure displayGridColumnWidth is an integer instead?

*** Bug 601948 has been marked as a duplicate of this bug. ***

Created attachment 147903 [details] [review]
[overview] Integer align workspace and pane positioning

We had previously been leaving the scaled workspace at a non-integral
position which didn't look obviously ugly, but now that we're
constraining the popup pane to the workspace size, we really need
to ensure that we're using integral positions here.

(In reply to comment #3)
> (From update of attachment 147701 [details] [review])
> >+            this._activeDisplayPane.actor.width = Math.floor(displayGridColumnWidth * 2);
> 
> but, seen in the context part of the diff:
> 
> >         this._dash.actor.set_size(displayGridColumnWidth, contentHeight);
> 
> which will presumably cause problems elsewhere? Should we just make sure
> displayGridColumnWidth is an integer instead?

Yeah, probably saner.  New patch attached.