xsettings added back to moblin applets

[Kyle Nitzsche]

Tracker bug for adding xsettings to moblin-settings-daemon in moblin-applets, as discussed in a few weekly IRC meetings.

This is probably needed to enable dynamically changing the gtk theme from gconf, as is done in Desktop.

[Kyle Nitzsche (knitzsche)]

I would like to request that the importance of this bug be changed to critical. Action items to implement this feature have been assigned for weeks in the weekly ubuntu mobile irc meeting. It is a gating function for dynamic gtkrc theme switching, and we need to have this working soon so that we can do subsequent development for a client. Thank you.

[ToddBrandt (todd-e-brandt)]

Kyle, I have those settings added back in but since we only have one theme it's difficult to test. Is there some sort of theme stress test you could send me to let me verify?

[Kyle Nitzsche (knitzsche)]

Todd,

In Desktop, the current gtk theme exists as the following gconf key: /desktop/gnome/interface/gtk_theme

Its value is the name of a theme directory here: /user/share/themes/<themeDir>

The default theme in mobile is "mobilebasic", which corresponds to this dir: /usr/share/themes/mobilebasic

In Desktop, if you manually reset the gconf key to an existing theme directory, the gtk rc files in it are automatically reloaded and gtk widgets and applications redraw accordingly.

The goal is identical behavior in mobile.

To test whether the changes you made enable such behavior, do the following in mobile:

 * copy recursively /usr/share/themes/mobilebasic to /usr/share/themes/testtheme

 * edit /usr/share/themes/testtheme/gtk-2.0/gtkrc

 * change this:
style "osso-color-themeing" = "osso-logical-colors" {
  fg[NORMAL] = @DefaultTextColor

 * to this:
style "osso-color-themeing" = "osso-logical-colors" {
  fg[NORMAL] = "#ff1000"

The result of this is that (some) widget foreground text is red. For example, launch date and time settings from the marquee and month and year are red. Also, the "Starting <app>" pop-up has red text.

So, with in a running target, modify the controlling gconf key (/desktop/gnome/interface/gtk_theme) to be "mobilebasic" (instead of its default value of "Human")

The test succeeds if gtk fonts (at least in cases described above) become red dynamically.

cheers.

[Kyle Nitzsche (knitzsche)]

One correction:
I said to test, modify the gconf key to be "mobilebasic". I meant modify it to be "testtheme", the theme in which you made the gtkrc file edit that turns fonts red.

[Kyle Nitzsche (knitzsche)]

We also need matchbox-window-manager compiled with xsettings support as a critical piece required for dynamic gtk theme switching.

[Loïc Minier (lool)]

m-w-m inherits xsettings support from libmatchbox (but needs a rebuild).

[Loïc Minier (lool)]

It seems matchbox xsettings support is actually mostly unrelated; it's useful to have it, but not required. It will listen to these settings:
    { "Net/ThemeName", XSET_THEME },
    { "MATCHBOX/THEME", XSET_THEME },
    { "MATCHBOX/CURSOR", XSET_CURSOR },
    { "MATCHBOX/TITLEBARS", XSET_TITLEBARS }, /* XXX Not implemeted */
    { "MATCHBOX/COMPOSITE", XSET_COMPOSITE },
    { "Gtk/CursorThemeName", XSET_CURSOR_THEME_NAME },
    { "Gtk/CursorThemeSize", XSET_CURSOR_THEME_SIZE },

The first one is likely interesting and the last two perhaps to some extent; we probably don't need the other ones.

Anyway, this shouldn't prevent switching themes for all practical purposes, it will just get us a nicer mwm.

[Kyle Nitzsche (knitzsche)]

xsettings settings is confirmed added back to moblin-applets 0.55jax1.

Happy to say that modifying /desktop/gnome/interface/gtk_theme gconf key manually to another valid gtk theme dir now causes auto-propagation of new gtkrc info to all gtk apps, which redraw themselves accordingly. (Note: this does not work reliably in xephyr+target.)

[Tony Espy (awe)]

Kyle confirmed in the previous comment that this was fixed in moblin-applets_0.55jax1. The fix was actually was made in version 0.55 which we modify ( hence the jax1 ). The version in our repos now is 0.55jax2.

As per yesterday's triage meeting, I'm marking this "fix committed".

[ToddBrandt (todd-e-brandt)]

Changed in moblin applets to fix committed

[cathyli (cathy-li)]

Verified in latest moblin-applets, the issue is fixed, close it.

[elvafu (elva-fu)]

assigned to m11

[Emmet Hikory (persia)]

Fixed in matchbox-window-manager as of 1.2-2

[Emmet Hikory (persia)]

FIxed in libmatchbox as of 1.9-4