newmanls / rofi-themes-collection

Themes Collection for Rofi Launcher
GNU General Public License v3.0
590 stars 49 forks source link

Fixed 'active' highlight not showing on every other item in the list. #15

Closed sslater11 closed 4 months ago

sslater11 commented 4 months ago

Hi, this is a fix for the 'active' highlight which doesn't always show. There's an option to have the the active colour change for odd and even numbers, but only one was showing. I couldn't fix it for the 'launchpad.rasi' file, the active highlight doesn't work at all on that one.

I've made it so all the odds and even list items get highlighted accurately.

In this example you'll see the orange text, that's how it should always look, but sometimes all the text is white. output

If you want to test this bug yourself, run this code which will spam you out with rofi. Just press Esc loads of times to quit :).

cd rofi-themes-collection/themes
theme="rounded-orange-dark.rasi"
echo -e "f\hellp there\nme-youz\flumpity\naaaaaa\nmore\nof\nthese\nyay" | rofi -dmenu -selected-row 1 -a 1 -theme $theme
echo -e "f\hellp there\nme-youz\flumpity\naaaaaa\nmore\nof\nthese\nyay" | rofi -dmenu -selected-row 2 -a 2 -theme $theme
echo -e "f\hellp there\nme-youz\flumpity\naaaaaa\nmore\nof\nthese\nyay" | rofi -dmenu -selected-row 3 -a 3 -theme $theme
echo -e "f\hellp there\nme-youz\flumpity\naaaaaa\nmore\nof\nthese\nyay" | rofi -dmenu -selected-row 4 -a 4 -theme $theme
echo -e "f\hellp there\nme-youz\flumpity\naaaaaa\nmore\nof\nthese\nyay" | rofi -dmenu -selected-row 5 -a 5 -theme $theme
echo -e "f\hellp there\nme-youz\flumpity\naaaaaa\nmore\nof\nthese\nyay" | rofi -dmenu -selected-row 6 -a 6 -theme $theme
echo -e "f\hellp there\nme-youz\flumpity\naaaaaa\nmore\nof\nthese\nyay" | rofi -dmenu -selected-row 7 -a 7 -theme $theme
echo -e "f\hellp there\nme-youz\flumpity\naaaaaa\nmore\nof\nthese\nyay" | rofi -dmenu -selected-row 8 -a 8 -theme $theme