Skip to content

Conversation

cichli
Copy link
Member

@cichli cichli commented Mar 20, 2015

Fixes #1033.

@vspinu
Copy link
Contributor

vspinu commented Mar 20, 2015

Confirmed. That fixes it. But it's strange as font-lock-face is designed to be used in font-lock modes instead of face.

@vspinu
Copy link
Contributor

vspinu commented Mar 20, 2015

Are there other usages of font-lock-face also broken or it only concerns buttons?

@cichli
Copy link
Member Author

cichli commented Mar 20, 2015

Yeah, I haven't dug in too deeply but this seems to be a button.el specific thing. As far as I can tell this is the only place where we're not using the default button face on buttons (we also use buttons in apropos, doc, and test modes, but I'm not seeing this specific issue in any of them).

bbatsov added a commit that referenced this pull request Mar 20, 2015
Use `face` text property for error buffer buttons
@bbatsov bbatsov merged commit fe7fc29 into clojure-emacs:master Mar 20, 2015
@bbatsov
Copy link
Member

bbatsov commented Mar 20, 2015

👍 Guess I overdid it when trying to fix the REPL font-locking. :-)

@cichli cichli deleted the button-face-text-property branch March 20, 2015 17:39
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

Successfully merging this pull request may close these issues.

Erroneous underlining in error buffer
3 participants