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] Modeline Background gets Broken Up #486

Closed
2 of 3 tasks
dylanjm opened this issue Oct 27, 2021 · 20 comments
Closed
2 of 3 tasks

[Bug] Modeline Background gets Broken Up #486

dylanjm opened this issue Oct 27, 2021 · 20 comments
Labels
bug Something isn't working

Comments

@dylanjm
Copy link

dylanjm commented Oct 27, 2021

Thank you for the bug report

  • I am using the latest version of doom-mode related packages.
  • I checked FAQ.
  • You may also try reproduce the issue using clean environment and minimal configurations with
    the command emacs -Q.

Bug description

Sometimes the modeline background gets broken up and doesn't extend across the buffer all the way.

Steps to reproduce

This is my configuration for doom-modeline:

(use-package doom-modeline
  :straight t
  :ghook ('after-init-hook)
  :config
  (general-setq doom-modeline-buffer-encoding t
                doom-modeline-checker-simple-format t
                doom-modeline-hud t
                doom-modeline-project-detection 'projectile))

This is my configuration for all-the-icons:

(use-package all-the-icons
  :straight t
  :commands (all-the-icons-icon-for-buffer)
  :config
  (general-setq all-the-icons-scale-factor 1)
  (general-pushnew '("\\.db$" all-the-icons-faicon
                     "database" :face all-the-icons-blue)
                   all-the-icons-icon-alist)
  (general-pushnew '("\\.edn$" all-the-icons-alltheicon
                     "clojure" :face all-the-icons-green)
                   all-the-icons-icon-alist))

Expected behavior

To not have the break in modeline.

OS

MacOS

Emacs Version

gccemacs-28

Emacs Configurations

https://github.com/dylanjm/teton/tree/master/config/emacs

Error callstack

No response

Anything else

image

image

Expected Behavior
image

@dylanjm dylanjm added the bug Something isn't working label Oct 27, 2021
@seagle0128
Copy link
Owner

Can you please describe the detailed steps to reproduce?

@dylanjm
Copy link
Author

dylanjm commented Oct 27, 2021

Hi, I don't mean to be difficult, but short of commenting out my config line by line until I narrow down the problem -- I don't see how I could provide a more reproducible example than what I've provided up above in my issue.

Please advise if there is anything I could do to narrow down this problem more.

@seagle0128
Copy link
Owner

Sorry, no idea.

@GorrillaRibs
Copy link

I'm seeing this too now, it looks like it might be related to emacs 29 & the new mode-line-active face? It didn't show up for me until I started using emacs from git, and seems fine in emacs 28.

@GorrillaRibs
Copy link

After a bit of testing, reverting this commit fixes the visual bug, is there any way to work around this?

@seagle0128
Copy link
Owner

@GorrillaRibs I think you should report to emacs-dev. The commit is stupid and broke the things...

@tshu-w
Copy link

tshu-w commented Dec 15, 2021

I have similar issue and fix with:

(set-face-attribute 'mode-line nil :inherit 'default)
(set-face-attribute 'mode-line-active nil :inherit 'mode-line)
(set-face-attribute 'mode-line-inactive nil :inherit 'mode-line)

@skyler544
Copy link

(set-face-attribute 'mode-line nil :inherit 'default)
(set-face-attribute 'mode-line-active nil :inherit 'mode-line)
(set-face-attribute 'mode-line-inactive nil :inherit 'mode-line)

this doesn't make any difference for me; I'm on the Emacs development branch. Minimal necessary configuration to reproduce:

(require 'package)
(add-to-list 'package-archives
             '("melpa" . "https://melpa.org/packages/") t)
(package-initialize)
(unless (package-installed-p 'use-package)
  (package-install 'use-package))
(eval-when-compile
  (require 'use-package))
(setq use-package-always-ensure t)

(use-package doom-modeline
  :config (doom-modeline-mode))

(use-package emacs
  :init
  (set-face-attribute 'mode-line nil :inherit 'default)
  (set-face-attribute 'mode-line-active nil :inherit 'mode-line)
  (set-face-attribute 'mode-line-inactive nil :inherit 'mode-line))

@tshu-w
Copy link

tshu-w commented Dec 23, 2021

Yes, I find this issue still happen when I switch to some other app and switch back on macOS. I thought it's emacs29 bug.

@GorrillaRibs
Copy link

GorrillaRibs commented Dec 23, 2021

I haven't gotten around to submitting a bug upstream yet, but for now building with this patch is working fine

modeline.patch.gz

updated, I had the patch inline but it was way too long

@seagle0128
Copy link
Owner

The latest Emacs 29 fixed the issue. I am closing the issue.

@skyler544
Copy link

The latest Emacs 29 fixed the issue. I am closing the issue.

Could you let me know which commit fixed this? I assumed that I would get the fix if I waited a few days and rebuilt, but since this comment I've rebuilt several times and the fix has yet to appear.
image

@seagle0128
Copy link
Owner

@skyler544
Copy link

The last commit I built from was yesterday: emacs-mirror/emacs@cbbe235

Seems like there was either a regression or the bug is caused by something else.

@seagle0128
Copy link
Owner

It seems it's a regression issue. You can check the faces of mode-line. I don't want to change before the functionalities are stable in 29.

@Retrodynen
Copy link

@skyler544 did you find a solution? i still have this issue on da0d598190c3337c4acb2adb7435b756c655af87

@skyler544
Copy link

@skyler544 did you find a solution? i still have this issue on da0d598190c3337c4acb2adb7435b756c655af87

I got sick of frequent problems while on the development branch and now that Emacs 28 is in pre-release I've switched to it and haven't had any issues. Neither my daily config or the minimal config I posted above show this bug on Emacs 28.

@Retrodynen
Copy link

ah ok, fair enough. thanks for the reply, i'll do the same this evening

@Retrodynen
Copy link

yeah seems it's an Emacs 29-specific issue :)

@seagle0128
Copy link
Owner

So I suggest to use 28 until 29 is stable.

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

6 participants