Merge lp://qastaging/~unity-2d-team/unity-2d/real-transparency into lp://qastaging/unity-2d/3.0

Proposed by Ugo Riboni
Status: Merged
Merged at revision: 586
Proposed branch: lp://qastaging/~unity-2d-team/unity-2d/real-transparency
Merge into: lp://qastaging/unity-2d/3.0
Diff against target: 647 lines (+174/-240)
17 files modified
debian/libunity-2d-private0.install (+1/-0)
launcher/Launcher.qml (+18/-2)
launcher/app/launcher.cpp (+1/-4)
launcher/app/launcherview.cpp (+8/-0)
libunity-2d-private/Unity2d/CMakeLists.txt (+3/-1)
libunity-2d-private/Unity2d/GnomeBackground.qml (+113/-0)
libunity-2d-private/Unity2d/qmldir (+2/-0)
libunity-2d-private/Unity2d/screeninfo.cpp (+5/-0)
libunity-2d-private/Unity2d/screeninfo.h (+4/-0)
libunity-2d-private/src/unity2dpanel.cpp (+8/-2)
libunity-2d-private/src/unity2dpanel.h (+1/-1)
places/GnomeBackground.qml (+0/-107)
places/app/dashdeclarativeview.cpp (+0/-6)
places/app/dashdeclarativeview.h (+0/-3)
places/dash.qml (+6/-5)
spread/GnomeBackground.qml (+0/-109)
spread/Workspace.qml (+4/-0)
To merge this branch: bzr merge lp://qastaging/~unity-2d-team/unity-2d/real-transparency
Reviewer Review Type Date Requested Status
Florian Boucault (community) Approve
Review via email: mp+62096@code.qastaging.launchpad.net

Commit message

[dash][launcher] Implement real transparency when there's a compositor running.

Description of the change

This branch implements real transparency in the launcher and dash when a compositing manager is active.

The opacity of the transparency may be off and need to be double checked.

Please also note that I put in some fixes in the GnomeBackground QML class, which now is not tied anymore to a specific view (it is still copied in two places, but it's easier to merge into a plugin when we decide to do that).

There's also a fix for performance, so that the fake background is only loaded when the compositor is not running.

Finally, please note that the bug regarding slideshow backgrounds (703574) is fixed by this patch only when compositing mode is running. The fake transparency code still has the issue.
I suppose it can be fixed in the fake transparency case by taking the screenshot of the Nautilus desktop window, but then it would need to look for damage events on that window and update, and I didn't have time to implement that solution yet.

To post a comment you must log in.
Revision history for this message
Florian Boucault (fboucault) wrote :

The panel exhibits visual artifacts. See http://people.canonical.com/~kaleo/transparency_bugs.png

review: Needs Fixing
Revision history for this message
Florian Boucault (fboucault) wrote :

The background of the launcher used to have a one pixel line on its right. It is important to the visual design.

review: Needs Fixing
Revision history for this message
Florian Boucault (fboucault) wrote :

The spread used to have a few pixels of separation between each workspace.

review: Needs Fixing
Revision history for this message
Florian Boucault (fboucault) wrote :

The spread performance is highly hampered with that patch. Going in and out of the spread is sluggish (quite more than before).

review: Needs Fixing
Revision history for this message
Ugo Riboni (uriboni) wrote :

The revisions I just pushed should have put back the space between the workspaces in the spread, and fixed the performance regression.

I also put back the border in the launcher. I'm not sure if the way I did it was the most efficient. I also thought of using a 1px wide Rectangle, but went for the tiled pixel for now. Should be easy to change if you think it's better the other way around.

Finally I could not reproduce the corruption in the panel. I am unsure what may cause it since as you can see I don't touch anything in the panel.

Revision history for this message
Florian Boucault (fboucault) wrote :

Everything is good to go now :)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: