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

doom-modeline disregards value of column-number-indicator-zero-based #62

Closed
nsemrau opened this issue Dec 7, 2018 · 4 comments
Closed
Labels
bug Something isn't working

Comments

@nsemrau
Copy link

nsemrau commented Dec 7, 2018

First of all: Thank you for this package. I basically tried all of the new "modern" mode-lines for Emacs over the recent years and usually went back to the classic mode-line quickly. Not so with doom-modeline; I like everything about it and I will stay with it as the true successor to the classic mode-line. :)

Emacs 26.1 introduced the variable column-number-indicator-zero-based. When set to nil, column-number-mode shows 1 instead of 0 when point is on the leftmost column in a line.

This works as expected with the classic mode-line. Starting Emacs with doom-modeline enabled, however, column-number-modes value of the leftmost column is 0, regardless of column-number-indicator-zero-baseds value.

I'm using Emacs 26.1 stable with the GTK3 toolkit, doom-modeline 20181206.1735 from MELPA with this setup:

(use-package doom-modeline
  :ensure t
  :hook (after-init . doom-modeline-init)
  :init (setq doom-modeline-icon nil))
@seagle0128 seagle0128 added the enhancement New feature or request label Dec 7, 2018
@seagle0128
Copy link
Owner

seagle0128 commented Dec 7, 2018

Hmm, I thought I has implemented it already. I tried some modelines but no lucky, except the vanilla mode-line. Seems a bug here, for both column-number-indicator-zero-based and mode-line-percent-position.

@seagle0128 seagle0128 added bug Something isn't working and removed enhancement New feature or request labels Dec 7, 2018
@seagle0128
Copy link
Owner

I realized it's from a PR and as designed. But I addressed it anyway. Please try.

@nsemrau
Copy link
Author

nsemrau commented Dec 8, 2018

doom-modeline now honors the value of column-number-indicator-zero-based and switches back and forth when I change the variable's value. Thank you for the instant fix. 👍

@seagle0128
Copy link
Owner

Welcome and enjoy!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants