genesis-devs team mailing list archive

[Frederik Elwert <frederik.elwert@xxxxxx>]

Hi!

Am Samstag, den 07.11.2009, 22:11 +0100 schrieb David Castellanos: 
> 2009/11/7 Frederik Elwert <frederik.elwert@xxxxxx>:
> >      * GTK yields a RuntimeWarning because the settings dialog and the
> >        server assistant are in the same ui file, but only signals for
> >        one are connected at a time. Glade allowed to load only one
> >        top-level widget from a glade file, but I think Builder always
> >        loads all. So should we split the two widgets in separate ui
> >        files, or just ignore the warning?
> 
> You're right. I don't remember where I read it, but it was advised to
> use only one top-level widget per file. For example take a look in MS
> sources, there are as many .ui files as top-level widgets. If you
> want, take for example the preferences_dialog.ui from MS and change
> the preference strings... I was careful with Gnome UI advices like
> element spacing and so.

I just pushed the changes to the gtkbuilder branch. I would be happy if
you could review the changes before I merge them into trunk.

You’re right, your dialog looks somewhat better regarding spacing etc.
But it turned out to be easier to just split the existing files. We
could look into more HIG compliance later on.

> >      * intltool-extract seems to have issues with ui files instead of
> >        glade files. This is why update-translations.sh currently fails.
> 
> Mmmm, I don't remember to have problems with this at all, I extracted
> the strings with the command: intltool-extract --type="gettext/glade"
> *.ui, and then xgettext... etc. If you want, I could take a look on
> this.

I think I got it. I just forgot to change all relevant files from glade
to ui. Should work now.

Cheers,
Frederik