UBUNTU_UI_TOOLKIT_THEMES_PATH should include a writable path so that users can install themes (or operators can have custom themes)

Also note that in Ambiance's MainViewStyle.qml, it hardcodes the theme setting as Ubuntu.Components.Themes.%1, where this may not be the path of a Custom theme (mine for example is UbuntuCustom.Themes.Whatever)

The env var can point anywhere, also to a writable spot. If that writable spot contains themes that derive from one of the basic themes, those will work properly. However if those themes are derived from other themes not installed to the default location, applying the theme will fail. For that this env variable is not enough. So we need an additional setting/env var pair which stores the additional paths the custom themes reside.

Zsombor,

When I tried to change the path to a writable spot and then derive from one of the default themes, this didn't work, as it was missing theming elements (like ProgressBarStyle, etc). Perhaps because I had used a different naming scheme? My theme lived in /custom/usr/share/themes/UbuntuCustom/Themes/Custom, with UBUNTU_UI_TOOLKIT_THEMES_PATH pointing to /custom/usr/share/themes, so my used theme was UbuntuCustom.Themes.Custom. It couldn't use the parent theme until I either included it in /custom/usr/share/themes, or made a symlink to it.

Chris,

Do you have any path which you know it will be used to store custom themes?

UBUNTU_UI_TOOLKIT_THEMES_PATH is only for development purposes, I don't think it should be used in production images. Instead we should have a set of paths themes may reside. The default is on the RO area, is the /custom/usr/share/themes the other one?

Zsombi,

At the moment the other one is /custom/usr/share/themes, but would it maybe make sense to have it reside somewhere writable in XDG_CONFIG_DIR?

Chris,

Is it XDG_CONFIG_DIR or XDG_CONFIG_DIRS env var? We could fetch that too, no question about. I haven't seen either of those set on the device...

Zsombi,

For themes I'd recommend XDG_DATA_DIRS. This maps to /usr/share/ generally and we add a path in /custom to it when we modify the phone image.

Fix committed into lp:ubuntu-ui-toolkit at revision None, scheduled for release in ubuntu-ui-toolkit, milestone Unknown

This bug was fixed in the package ubuntu-ui-toolkit - 0.1.46+14.04.20131105.1-0ubuntu1

---------------
ubuntu-ui-toolkit (0.1.46+14.04.20131105.1-0ubuntu1) trusty; urgency=low

  [ Albert Astals ]
  * Minor optimzations Add const & to foreach to save copies of
    "complex" types Remove initializations of QStrings to "" since they
    do nothing Return QString instead of "" since its faster Pass
    "complex" types as const & instead of copy to functions Not an
    optimization: Added const qualifier to two functions .

  [ Lars Uebernickel ]
  * Remove GIconProvider Use urls for file and data icons and the theme
    image provider (image://theme/) for themed icons.

  [ Kaleo ]
  * [visual design] Updated tabs chevron asset.

  [ Christian Dywan ]
  * Install translations for UI toolkit and gallery. (LP: #1239627)
  * Support checkbox used with an action.
  * Install autopilot modules in qmake not debian packaging.
  * Drop the $$ from system calls where output isn't needed.
  * Look for locale folder as per XDG basedir spec and rewrite test The
    new test case covers the whole range of changing languages, checking
    the strings from both QML and C++ and binding to a folder. (LP:
    #1175726, #1233071)
  * Define window within OrientationHelper to avoid a race condition.
    (LP: #1239760)

  [ Zsombor Egri ]
  * Support for taking themes from XDG_DATA_DIRS as well as from
    ~/.local/share. These paths are also added as import paths.
    UBUNTU_UI_TOOLKIT_THEMES_PATH overrules these import paths, and can
    have more than one path specified, each separated with column. (LP:
    #1221813)
  * Warnings treated as errors in all types of builds (debug or
    release). (LP: #1246290)
  * Introducing topmostItem property to drive whether InverseMouseArea
    should filter sensingArea events or to stay in the normal z-
    order/stack order of its parent. (LP: #1240460)

  [ Leo Arias ]
  * Added the go back method to the autopilot emulators. (LP: #1239751)
  * On the autopilot emulator, drag the toolbar to its full height. (LP:
    #1235060)
  * Initially show toolbar, but automatically hide after timeout. (LP:
    #1207369)
  * Added the change_state method to the CheckBox autopilot emulator.
    Cleaned up the autopilot gallery toggles tests.
  * Added the objectNames to the ComposerSheet buttons. (LP: #1244518)

  [ tpeeters ]
  * Update CrossFadeImage API so that sourceSize can be set by the
    applications. CHANGED in CrossFadeImage: readonly property size
    sourceSize TO property size sourceSize. (LP: #1227783)
  * Initially show toolbar, but automatically hide after timeout. (LP:
    #1207369)
  * Put tab bar in selection mode when apps are launched. (LP: #1223597)
  * Close tabbar and toolbar when user interacts with app contents. (LP:
    #1223604)

  [ Nicolas d'Offay ]
  * Added multiSelection bool and supporting code to OptionSelector and
    ItemSelector. This turns either component into a multiple choice
    selector. Fixed linked bugs. (LP: #1231936)

  [ Renato Araujo Oliveira Filho ]
  * Implemented emulator for swipe to remove. (LP: #1236464)

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 820
 -- Ubu...