Code review comment for lp:~dandrader/qtmir/dontOccludeFreshSurfaces

Revision history for this message
Gerry Boland (gerboland) wrote :

+ // TODO: Make it configurable, exposing it as a QML property to shell.
I'm not keen on this, as it pushes the responsibility to the shell. How can shell ever get this value right? IMO It's the apps problem that it isn't drawing a perfect first frame - and I don't understand why Dash/Qt is failing to do this.

However in the meantime, this fix does do the job. Let's use it for now

review: Approve

« Back to merge proposal