In treebrowser/src/treebrowser.c:

> +        gtk_style_context_add_provider_for_screen (screen, GTK_STYLE_PROVIDER (provider),
> +                                                   GTK_STYLE_PROVIDER_PRIORITY_APPLICATION);
> +        if(!is_dir)
> +        {
> +            gtk_css_provider_load_from_data (GTK_CSS_PROVIDER(provider),
> +                                             " #addrbar {\n"
> +                                             "   background: red;\n"
> +                                             "}\n", -1, NULL);
> +        }
> +        else
> +        {
> +            gtk_css_provider_load_from_data (GTK_CSS_PROVIDER(provider),
> +                                             " #addrbar {\n"
> +                                             "   background: white;\n"
> +                                             "}\n", -1, NULL);
> +        }

and actually this isn't a proper port of what the original code did in GTK2: it used to set both background and foreground when is_dir evaluated to false, and here you change only the background, but to a different color depending on the is_dir value. That's not good, as you change background but not foreground, so it might very well be unreadable (imagine white on white for a dark theme).

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.