On Thu, 2012-06-21 at 20:03 -0500, Travis Carter wrote: > I've already submitted a patch that isn't as well formed to the bug > listing. Sorry. This was my first patch to any project and I should > have checked a little better on the preferred submission method. The > one attached here should be better.
Looks good. So... First up, can you add yourself to http://wiki.documentfoundation.org/Development/Developers and link to a mail that states that your submission is under our preferred MPL/LGPLv3+ license combo. Most seem to go for a simple "all my patches unless I say otherwise" Second up, I wonder if we should do the "save position" thing *both* in the dtor and in the ::Ok handler that's called from apply. I mean, at the moment we seem to store which is the open tab and the position/size of the dialog on cancel/close too, do we want to continue doing that ? C. _______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice