vinceliuice / Graphite-gtk-theme

Graphite gtk theme
GNU General Public License v3.0
1.06k stars 52 forks source link

Issue installing Theme Variants with "-t all" argument #150

Closed beansh closed 4 months ago

beansh commented 4 months ago

Looks like the recent change in the install_theme() function, specifically changing ${themes[@]} to ${themes[0]}, has introduced an issue with the installation of theme variants when using the -t all argument.

When running ./install.sh -t all only the default theme variant is installed. In order to install any other theme variant, the argument -t blue for example must be explicitly given. Adding multiple colors to the argument ignores all but the first.

The loop does not iterate over all theme variants correctly. This change effects the AUR graphite-gtk-git package as all theme variants are no longer installed or removed when the package is upgraded.

The commit 8c61bbd2f4f427cabaceb1f373b5218db77e048e introduced the issue.