Re: Cheese patches

[Frederic Peters <fpeters gnome org>]

Bastien Nocera wrote:

> The largest patch ("Don't crop the video preview") uses a renamed
> MxAspectFrame (currently under review in upstream Clutter), which was
> already used within an Intel internal project. It works well in my
> testing.

But it introduces a dependency on an unspecified version of mx; will
this work with the stable 1.0 serie of mx, or does it require 1.1?

And mx is not even part of our stack...


> "Make the flash not hide the gnome-shell panel" follows advice by Dan
> Winship (IIRC) not put the flash over the shell panel. The majority of
> the code is a cut'n'paste of code that's been used in Totem, and
> gnome-desktop. Works well in my testing as well.

No objection for this one.


        Frederic