Re: [PATCH] Give Macro recorder separate option, apart from Experimental features

On Mon, 2012-06-18 at 07:53 +0200, Cor Nouws wrote:
> >     Incidentally - if you want to get this into 3.6 as a late feature you
> > need to be drumming up support for another 2x reviews quickly ;-) - we
> > want to get that lot in by Monday.
>
> Ah, if possible, that would be nice.
> I see a + 1 from Olivier already, so if one other developer
> has a little 

        So - it's good to have the patch included :-)

        However, it should be noted that by submitting this patch you (and
Olivier?) are morally volunteering ;-) to fix any macro recording /
playback bugs that get filed ;-) There is a reason that the feature was
disabled in the 1st instance.

        The text "(limited)" seems sensible; I'd prefer 'experimental' to share
the connotation of unfinished, not-fully-working, un-supported etc. in
that text label personally. The configuration comment is not shown in
the UI but captures some of that.

        Anyhow - hopefully it's not a big problem; disabled by default & it is
probably in a different category to the other experimental features that
are riskier so it prolly still makes sense.

        I'm also slightly amazed you managed to find vertical space in that
dialog to cram that thing into; did you test with the worst-case setup
there of tons of those options being visible ? IIRC that happens on
Windows.

        Petr - I guess we should make a mental note to patch this out of SUSE
3.6 builds as/when :-)

        Thanks !

                Michael.

-- 
michael.meeks@suse.com  <><, Pseudo Engineer, itinerant idiot