Re: [Libreoffice] [PUSHED] Introduce HideDisabledMenuItems style setting

2011-04-28 Thread Lubos Lunak
On Thursday 28 of April 2011, Michael Meeks wrote:
> Hi Christian,
>
> On Wed, 2011-04-27 at 11:13 +, Christian Dywan wrote:
> > I updated the patch and attached a second one to get rid
> > of UpdateApplicationSettings which for no good reason tries
> > to flip the same switch in a very misleading way.
>
>   Thanks for the nice patches, which I just pushed.
>
>   So sorry it took a week to get this, IMHO helpful and sensible cleanup
> pushed.

 I've reverted the KDE parts of the patch as they do not match KDE behavior.

-- 
 Lubos Lunak
 l.lu...@suse.cz
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] [PUSHED] Introduce HideDisabledMenuItems style setting

2011-04-28 Thread Christian Lohmaier
Hi Michael, *,

On Thu, Apr 28, 2011 at 12:19 PM, Michael Meeks
 wrote:
> On Wed, 2011-04-27 at 11:13 +, Christian Dywan wrote:
>> I updated the patch and attached a second one to get rid
>> of UpdateApplicationSettings which for no good reason tries
>> to flip the same switch in a very misleading way.
>
>        Thanks for the nice patches, which I just pushed.

OK .- as apparently you understand what the patch does - would you
then please describe its effects to Christoph, Allen and me?

Christian Dywan's "clarification mail"  didn't clear anything up for me.

ciao
Christian
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice] [PUSHED] Introduce HideDisabledMenuItems style setting

2011-04-28 Thread Michael Meeks
Hi Christian,

On Wed, 2011-04-27 at 11:13 +, Christian Dywan wrote:
> I updated the patch and attached a second one to get rid
> of UpdateApplicationSettings which for no good reason tries
> to flip the same switch in a very misleading way.

Thanks for the nice patches, which I just pushed.

So sorry it took a week to get this, IMHO helpful and sensible cleanup
pushed.

Personally, I dislike the Application::SetSettings stuff - it looks
like a disaster area to me when it comes to merging user / system etc.
settings.

I'd love to re-factor & re-implement the remaining five uses of that
outside of vcl/ - then perhaps we could get rid of all those
'SetDoubleClickTime', 'SetStartDragWidth' etc. methods which surely
should be internal impl. details of VCL.

Anyhow - thanks again, and looking forward to your next cleanup.

ATB,

Michael.

-- 
 michael.me...@novell.com  <><, Pseudo Engineer, itinerant idiot


___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice