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

Michael Meeks wrote (19-06-12 12:28)

>         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.

Cool suggestion - Olivier, don't you think so ;-)

>         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&

Yep, by choice.

> it is probably in a different category to the other experimental
> features that are riskier so it prolly still makes sense.

Thanks for recognising that :-)

>         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.

Since I'm not building on Windows, it's not yet tested in a situation 
where all options were for sure visible. However I as reference I took a 
situation that I saw some time ago, and that gave me the impression 
that... But no, not fully sure yet. Will look asap when there is a build 
having the patch.
There's also the small problem that the write/read from the 
configuration is not yet functional (see my initial mail).

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

So there you guys will be taking the moral responsibility to support the 
poor users loosing work while having the undefined pool 'experimental' 
activated, just to record a partly broken macro ... :-D

>         Thanks !

Thanks for your support, Michael!

--
 - Cor
 - http://nl.libreoffice.org