[PATCH] various .ui fixes caught by linter tool
Michael Meeks
michael.meeks at suse.com
Mon Mar 18 04:45:44 PDT 2013
On Mon, 2013-03-18 at 12:01 +0100, Lubos Lunak wrote:
> > On Fri, 2013-03-15 at 18:32 +0000, Jack Leigh (via Code Review) wrote:
> > > various .ui fixes caught by linter tool
> >
> > Sounds really interesting :-) can we make the tool part of our build
> > process somehow ?
>
> And do we need such a tool at all? If Glade is not capable of providing
> sensible defaults for spacing and similar features, cannot our code for
> loading the .ui files handle that?
Not sure it's -that- easy, we can use the same widgets with different
padding around the place and/or may want to tweak that.
Either way - IIRC we also want the linter to ensure that all strings
are marked translated to avoid the problems we had around 4.0.0 here.
Jack - I guess a good place to put the lint tool might be in the
install rule for the .ui files:
solenv/gbuild/UI.mk
would be the place to add that; though it's not terribly clear
how/where - presumably the:
$(call gb_Package_Package_internal,$(1)_ui,$(SRCDIR))
piece does something like that,
Thanks !
Michael.
--
michael.meeks at suse.com <><, Pseudo Engineer, itinerant idiot
More information about the LibreOffice
mailing list