Code review comment for lp:~mc-return/compiz/compiz.merge-fix1099100-thumbnail-title-text-issues.0
Revision history for this message
| Sam Spilsbury (smspillaz) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
bbfa235
(Get the code!)
This looks fine code wise.
I'm a bit concerned as to how things will look once we start requiring that the text background be rendered in addition to the text. My understanding was that we were meant to render the thumbnail background (window) and then render the text on top of that. What was the reason that we couldn't do that? If it was that there was really a "hole" where the text was, then this sounds like a more fundamental problem with the rendering of the thumbnail background itself, and we shouldn't cover it up by rendering a text background on top of it.
Could you post a screenshot to see how it looks?