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

cider-apropos's font-locking is messed up #1565

Closed
bbatsov opened this issue Feb 14, 2016 · 4 comments
Closed

cider-apropos's font-locking is messed up #1565

bbatsov opened this issue Feb 14, 2016 · 4 comments
Labels

Comments

@bbatsov
Copy link
Member

bbatsov commented Feb 14, 2016

See this:

image

The offending code is here:

(defun cider-show-apropos (summary results query docs-p)
  "Show SUMMARY and RESULTS for QUERY in a pop-up buffer, formatted for DOCS-P."
  (with-current-buffer (cider-popup-buffer cider-apropos-buffer t)
    (let ((inhibit-read-only t))
      (set-syntax-table clojure-mode-syntax-table)
      (apropos-mode)
      (cider-mode)
      (if (boundp 'header-line-format)
          (setq-local header-line-format summary)
        (insert summary "\n\n"))
      (dolist (result results)
        (cider-apropos-result result query docs-p))
      (goto-char (point-min)))))

Not sure why the syntax table is used here. Enabling cider-mode also seems redundant.

@bbatsov bbatsov added the bug label Feb 14, 2016
@bbatsov
Copy link
Member Author

bbatsov commented Feb 15, 2016

@jeffvalk
Copy link
Contributor

Mainly for navigating/copy-and-pasting code examples that are embedded in docstrings. Has the font locking always been like this? I don't recall that...

@bbatsov
Copy link
Member Author

bbatsov commented Feb 15, 2016

Don't recall. But I'm pretty sure dropping this and disabling cider-mode would fix the font-locking, Probably the addition of dynamic font-locking messed this up.

@bbatsov
Copy link
Member Author

bbatsov commented Feb 15, 2016

This also fixed one more annoying problem - I'd get prompted about a comment delimiter when using cider-apropos (I'm guessing this was something to do with the syntax table).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants