Code review comment for lp:~heathbar/switchboard-plug-pantheon-shell/fix-1036271

Revision history for this message
Rico Tzschichholz (ricotz) wrote :

Great to see this included.
Although I want to mention there are some drawbacks. It includes regular files rather than only non-hidden directories. Also there are only two directories supported ~/.local/share/plank/themes and %PREFIX%/share/plank/themes
Sorting the list alphabetically would be useful too.

« Back to merge proposal