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

Bug: nano interferes with web-mode (TSX) #122

Open
hermann-p opened this issue Jun 8, 2022 · 13 comments
Open

Bug: nano interferes with web-mode (TSX) #122

hermann-p opened this issue Jun 8, 2022 · 13 comments

Comments

@hermann-p
Copy link

When switching to web-mode, it breaks with Error (prog-mode-hook): Error running hook "highlight-indent-guides-mode" because: (wrong-number-of-arguments #<subr quote> 0) and stays in fundamental mode.

Reproduce:

  • emacs -q -l nano.el
  • install web-mode with install-package
  • open up a TSX-file (did not test html ones yet)
  • M-x web-mode

Let me know if I can help debugging, but remember my elisp-fu is near to non-existent.

@rougier
Copy link
Owner

rougier commented Jun 21, 2022

Could you try to use nano-modeline instead (which is on ELPA)?

@hermann-p
Copy link
Author

Not sure what you mean? Instead of what?

@rougier
Copy link
Owner

rougier commented Jun 23, 2022

Sorry for not being clear. There is now a dedicated package (nano-modeline) that is available on ELPA. You can install it from Emacs and then test is by typing M-x: nano-modeline (from a "regular" Emacs session, not nano).

@hermann-p
Copy link
Author

hermann-p commented Jun 23, 2022

I got that part, just was confused about the "instead" part. So just the modeline instead of the whole nano package.

  • open a fresh Emacs emacs -q
  • installed nano-modeline and web-mode
  • opened the same .tsx-file as before
  • M-x nano-modeline
  • M-x web-mode

Works just fine. Works also if I load the modeline before opening the file.

@rougier
Copy link
Owner

rougier commented Jul 11, 2022

Sorry for late answer. Does that mean it is working properly now?

@hermann-p
Copy link
Author

hermann-p commented Jul 12, 2022

It means I can run nano-modeline, but still not nano in web-mode. Should I try updating nano-emacs? I don't see any recent commits, though.

@rougier
Copy link
Owner

rougier commented Jul 13, 2022

I need to update nano-emacs to reflect the change but the story is that I split it in independent packages (nano-theme and nano-modeline) such that people are free to use this or that part. In the meantime, I've also developped several packages that go along the idea of nano emacs. You can have a look at my configuration here: https://github.com/rougier/dotemacs

@hermann-p
Copy link
Author

Thank you for your time, and please don't feel pushed.
I am aware of the other packages, and I mostly use doom emacs with nano-theme.

On a side note: I feel like the decluttering of colours together with the font hints removes a lot of distraction while still adding visual structure. It helps to focus on complex source code, so, great work!

@rougier
Copy link
Owner

rougier commented Jul 25, 2022

Thanks. Coming back to your problem, does that mean that nano-theme + nano-modeline works as expected now?

@nhunzaker
Copy link

I was able to resolve the issue by commenting out this line:

(setq font-lock-maximum-decoration nil)

@rougier
Copy link
Owner

rougier commented Feb 14, 2023

Oh, that's surprising and I wonder how you found the problem. Do you want to make a PR (to just comment it by default) ?

@nhunzaker
Copy link

Oh, that's surprising and I wonder how you found the problem.

When it stopped working, I just started poking around with the settings to see how all of this actually worked and suspected this might be a font locking issue.

Oh, that's surprising and I wonder how you found the problem.

Sure thing!

@rougier
Copy link
Owner

rougier commented Feb 27, 2023

Do you want to make a PR?

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