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

Disable display-line-numbers-mode #182

Merged
merged 1 commit into from
Feb 25, 2020
Merged

Conversation

zjp
Copy link
Contributor

@zjp zjp commented Aug 18, 2019

This is a short onetwo-liner commit to add support for hiding line numbers by default for users of global-display-line-numbers-mode, the way they are hidden for users of global-linum-mode. It checks the major version first, since the new mode was added in Emacs 26.

@jcs090218
Copy link
Member

There are few problems with this PR.

  1. There are something wrong with CI, maybe you should fix it.
  2. Use when if you only have one condition.

@zjp
Copy link
Contributor Author

zjp commented Feb 15, 2020

It looks like now there's an issue with the final step that isn't due to this PR in particular.

@jcs090218
Copy link
Member

Yeah, I think you could just update the branch to the latest commit to resolve this kind of issue. Anyway, I am good with this kind of PR. Thanks!

One last minor issue.

  • On line 59, indentation should be 4 not 6.

…-mode (hiding line numbers in the dashboard by default)
@zjp
Copy link
Contributor Author

zjp commented Feb 23, 2020

I have rebased on master and fixed spacing. I'm not sure why test-emacs-25 is failing; when a condition fails the following elisp isn't supposed to be evaluated, so Emacs 25 shouldn't even know that symbol is there. Is this not the case for when?

@jcs090218
Copy link
Member

@zjp I don't think the when operator shouldn't do to anything like this. The log says this

emacs --batch -q -l .emacs/init.el
Contacting host: elpa.gnu.org:80
Failed to download `melpa' archive.
Package `use-package-' is unavailable
Makefile:12: recipe for target '.elpa' failed
make: *** [.elpa] Error 255

I think we could simply just rerun the test. @JesusMtnez What do you think? 😕

Thanks!

@JesusMtnez JesusMtnez changed the title Extend existing behavior of linum-mode to display-line-numbers-mode Disable display-line-numbers-model Feb 25, 2020
@JesusMtnez JesusMtnez merged commit df49817 into emacs-dashboard:master Feb 25, 2020
@JesusMtnez JesusMtnez changed the title Disable display-line-numbers-model Disable display-line-numbers-mode Feb 25, 2020
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.

None yet

3 participants