Comment 2 for bug 877012

Revision history for this message
John Bender (johnbender) wrote :

Attached is a working patch for this issue which assigns a screen value to the instantiated Unity2dPanel which is then used to determine the panel's geometry in place of the widget itself.

A few notes:

1. this patch was created against a clean branch from lp:ubuntu/oneiric/unity-2d, so I'm not clear if it will apply cleanly to the latest precise branch/dev. I'm happy to look into it.
2. the default value for m_screen might be better as -1 since the QT docs suggest that the screenGeometry will default in that case ( http://doc.qt.nokia.com/latest/qdesktopwidget.html#screenGeometry )
3. it's only been tested on my desktop setup and not the desktop environment this will most likely be run under ( gnome fallback? )

I've done my best to adhear to the coding guidelines I found in the repository, but any and all feedback is welcome/greatly appreciated.