Date: prev next · Thread: first prev next last
2011 Archives by date, by thread · List index


On Fri, Mar 25, 2011 at 08:48:25PM +0000, Michael Meeks <michael.meeks@novell.com> wrote:
      Looks lovely to me - though I guess it might be nice to just pick a
large number for RET_APPLY  and not put that in vcl; that seems to be
how others do it eg.

sfx2/inc/docvor.hxx:#define RET_EDIT_STYLE       100
sfx2/inc/sfx2/new.hxx:#define RET_TEMPLATE_LOAD       100

      etc. if we want to clean this up - we should add a:

#define RET_CUSTOM_START 100

      or something, to the vcl header, and then use that (I think)

Ah, I didn't see it. We talked about this with Cedric and he explained
an idea which avoids any modification in vcl, so I'm not planning to
push this version, a better one will come soon.

The main problem is not with the modification, but if you move the
dialog, click Apply, then its position will be reset, which is ugly.

Also, I can change the indentation in sw/source/ui/app/docst.cxx, I
didn't do it yet for easier review.
..
What's your opinion, OK to push?

      With the above tweak I'm enthusiastic - nice patch ! :-) but ... why
are you (Mr Awesome) doing 'easy' hacks ? :-) I guess for each one you
fix, you need to research & create one more ;->

Partly because of GSoC - it's listed on the wiki page that students
should solve one before applying. Partly as this touches GUI code -
which I never did before, so it amuses me. :)

Attachment: pgpb7kqU9pcbG.pgp
Description: PGP signature


Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.