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

Flx faces have no underline under theme sanityinc-tomorrow-night #32

Closed
goofansu opened this issue Jul 23, 2013 · 5 comments
Closed

Flx faces have no underline under theme sanityinc-tomorrow-night #32

goofansu opened this issue Jul 23, 2013 · 5 comments

Comments

@goofansu
Copy link

No description provided.

@lewang
Copy link
Owner

lewang commented Jul 23, 2013

That hasn't changed. Can you check that your version of flx-ido is the same as on master branch?

@oscarfv
Copy link

oscarfv commented Jul 23, 2013

flx just defines a face that inherits from another one. Current theme
can redefine flx-highlight-face. It's not flx's fault that your theme
effectively removes the underline attribute from flx-highlight-face.

Here, with solarized-dark theme, no underscores are shown. As soon as I
disable the theme they appear again.

@bbatsov
Copy link
Contributor

bbatsov commented Jul 27, 2013

@goofansu This has nothing to do with flx - it's related to the theme you're using. I guess @purcell doesn't like the underlines - I've also removed them from zenburn and solarized.

@lewang I guess you should close this issue.

@purcell
Copy link

purcell commented Jul 27, 2013

Agreed, that was my choice. But in fact someone submitted a fix for the faces yesterday and I've merged them, so if you grab the latest Tomorrow themes package from MELPA, the flx highlight face should be yellow and bold. (But still no underline!)

@goofansu
Copy link
Author

@purcell
@bbatsov

Thank you for the reply

Kind Regards,

goofansu

在 2013年7月28日星期日,上午2:01,Steve Purcell 写道:

Agreed, that was my choice. But in fact someone submitted a fix for the faces yesterday and I've merged them, so if you grab the latest Tomorrow themes package from MELPA, the flx highlight face should be yellow and bold. (But still no underline!)


Reply to this email directly or view it on GitHub (#32 (comment)).

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

5 participants