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

Timo Aaltonen (tjaalton) wrote :

On Fri, 30 Jan 2009, Paul Sladen wrote:

> On Thu, 29 Jan 2009, Steve Langasek wrote:
>> On Thu, Jan 29, 2009 at 05:09:40PM -0000, Alberto Milone wrote:
>>> have learnt to deal with X freezes by using Ctrl+Alt+Backspace.
>
> Ctrl-Alt-Backspace is widely offered in help guidance:
>
> http://www.google.com/search?q=site:help.ubuntu.com+ctrl-alt-backspace (271 results)

And it's still available for those that need it.

>> e.g., by requiring you to hold down Ctrl+Alt+Bksp for 5 seconds before
>
> This is the most sensible suggestion I've heard.
>
> Alberto: would you will to offer up a patch that forces a hold-for-5-seconds
> requirement with no other keys being pressed in the meantime?
>
> This would:
> * fix the root problem
> * without breaking the currently defined behaviour
> * without needing a configuration option

But it _does_ break the currently defined behaviour, depending on the
implemention.. (hold it down or keep hitting it N times). So the
documentation would have to be fixed anyway..

Remember that we'd need to keep the patch "forever".

t

« Back to merge proposal