[compiz] [PATCH] Add opacity limits

[Erkin Bahceci]

On 7/28/07, Kristian Lyngstøl <kristian at bohemians.org> wrote:
> On 7/28/07, Erkin Bahceci <erkinbah at gmail.com> wrote:
> > This patch adds appropriate limits (1-100) to opacity_values in core
> > options. Minimum is 1 to be consistent with the opacity changing
> > action and because it doesn't make sense to have an invisible window
> > that is still there.
>
> The use cases may not be many, but you could say that for 1% opacity
> too. I don't think it should be up to us to say that there aren't any
> use cases when there's no technical reason not to allow it.
>
> Maybe change the opacity changing action to 0 too? Temporarily making
> a window invisible doesn't sound too unreasonable.

You can make it 5 percent if 1 is too low. 0 is not a good idea
because if a user sets 0 as opacity value (probably unintentionally),
there will be invisible windows around that will prevent interaction
with windows behind them, which will clearly look like a bug. Why
would you want an invisible window that blocks interaction with the
windows behind it?

Anyway, the point of the patch was to have some limits rather than none.

- Erkin