Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Font of active tab in tab bar is always white #1326

Closed
pszalanski opened this issue May 14, 2018 · 6 comments
Closed

Font of active tab in tab bar is always white #1326

pszalanski opened this issue May 14, 2018 · 6 comments

Comments

@pszalanski
Copy link

The theme selection works fine for me, thanks for that.

However, no matter what theme I have selected, the active tab has a white font color, which makes it illegible on most of my themes, especially my system theme, which is Ambiance on Ubuntu.

See here:
screenshot from 2018-05-14 08-26-18

@gsemet
Copy link
Member

gsemet commented May 14, 2018

which version of guake are you under? Have you tried with latest version from HEAD?

@pszalanski
Copy link
Author

It was checked out yesterday.

@abcarroll
Copy link

abcarroll commented May 15, 2018

Having the same thing here. Had to downgrade to the version supplied by my distro's pkg manager, 3.2.2 edit: 3.0.5, and it stopped.

@pszalanski
Copy link
Author

I guess this was caused by the following commit: 21cf658

@abcarroll
Copy link

abcarroll commented May 17, 2018

Verified this. You can temporarily fix this by changing the css_data = assignment, around line 421 of guake/guake_app.py, as below:

        css_data = dedent(
            """
            .tab {{
              color: #aaaaaa;
            }}
            
            .tab:checked {{
              color: #000000;
              background-color: {};
            }}
            """.format(selected_bg_color)
        ).encode()

(of course, this would be just as "wrong" if you select a dark gtk theme)

If I understood the intent of 21cf658 a little better, I would attempt a real patch, but I just don't understand what the commit was trying to accomplish.

@gsemet
Copy link
Member

gsemet commented May 17, 2018

I can reproduce it on latest. I'll see if I can fix it asap

gsemet added a commit that referenced this issue May 17, 2018
fix #1326

Signed-off-by: Gaetan Semet <gaetan@xeberon.net>
gsemet added a commit that referenced this issue May 17, 2018
fix #1326

Signed-off-by: Gaetan Semet <gaetan@xeberon.net>
gsemet added a commit that referenced this issue May 17, 2018
fix #1326

Signed-off-by: Gaetan Semet <gaetan@xeberon.net>
@gsemet gsemet closed this as completed in 2908357 May 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants