seagle0128 / doom-modeline

A fancy and fast mode-line inspired by minimalism design.
https://seagle0128.github.io/doom-modeline/
GNU General Public License v3.0
1.29k stars 159 forks source link

[Bug] Null vertical padding after update #656

Closed david-alvarez-rosa closed 1 year ago

david-alvarez-rosa commented 1 year ago

Thank you for the bug report

Bug description

After update to Emacs 29, and update of doom-modeline package, I see a null vertical padding in the mode line.

See the screenshot for reference.

Null vertical padding mode line screenshot

Steps to reproduce

Just open Emacs.

Expected behavior

Expected behavior is to have some kind of vertical padding, so the mode line looks nice.

Emacs Configurations

Vanilla.

georgek commented 1 year ago

Did you set doom-modeline-height?

david-alvarez-rosa commented 1 year ago

Nope, @georgek, I did not set the variable. I'm not using it as I want the height of doom-modeline to adapt dynamically whenever I'm changing the font-size (using https://github.com/purcell/default-text-scale package).

seagle0128 commented 1 year ago

@david-alvarez-rosa Try (setq doom-modeline-height 21).

david-alvarez-rosa commented 1 year ago

Thank you @seagle0128 that works when I start my Emacs:

Doom modeline large

However, when I enlarge the text (using https://github.com/purcell/default-text-scale package), it still doesn't look:

Doom modeline normal

Perfect state for me would be to define the vertical padding / margin instead of the height, so it adapts dynamically. But it's a problem actually, so what you mentioned helps. Thanks!

seagle0128 commented 1 year ago

The padding is implemented with bar image actually. The height was changed to (+ (frame-char-height) 4) recently. You can increase the height if you like paddings.