Code review comment for lp:~albertomilone/gnome-control-center/randr-virtual

Bryce Harrington (bryce) wrote :

On Thu, Jan 29, 2009 at 07:03:18PM +0100, Alberto Milone wrote:
> > > On Thursday 29 January 2009 17:48:37 Mark Shuttleworth wrote:
> > >> I very much appreciate Alberto's work in implementing this, but would
> > >> like to test the decision against our commitment to ease of use. I think
> > >> enabling or disabling this feature in X is an expert option. I think we
> > >> can choose to enable it, or disable it, and of course we can allow
> > >> experts to change that through a config file or through a rebuild, but
> > >> this capability *should not* be exposed to end-users as a GUI option. It
> > >> will create more confusion than it will solve.
> In any case this change is definitely something worth mentioning in
> the release notes together with some easy steps to re-enable this
> functionality.

Yes, mentioning it in the release notes is a good idea.

I think this emphasizes that our original plan of putting this in the
(expert user oriented) xorg.conf options editor was a better

You know, perhaps the 'dontzap' command line tool alone may be
sufficient to appease users upset by this change. It will at least be
relatively easy to document and a lot easier to maintain long term than
a gnome-control-center patch.


« Back to merge proposal