Code review comment for lp:~ksamak/compiz/add_startup_option_to_negative

Revision history for this message
ksamak (ksamak) wrote :

> > optionSetActivateAtStartupNotify (boost::bind (&NegScreen::optionChanged,
> this,
> > _1, _2));
>
> This is also incorrect - it will cause all windows to be toggled whenever the
> option is changed, not just on startup.

Yes, this is a suggestion made by 3v1n0, that when the user switches this option in ccsm, he sees a immediate reaction. This ensures the user that changes have been taken into account
This switches all windows immediately indeed. Though i can think of no case where it is irrelevant.

« Back to merge proposal