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

Work around a bug where the tab-line isn't taken into account #115

Merged
merged 1 commit into from
Jan 16, 2025

Conversation

Stebalien
Copy link
Contributor

In Emacs before version 31, window-inside-absolute-pixel-edges returns the wrong y-offset when the per-buffer tab-line is enabled. The tab-line is taken into account when computing the window's height, so we adjust the y-offset after computing the height.

Fixed upstream in https://debbugs.gnu.org/cgi/bugreport.cgi?bug=75576.

In Emacs before version 31, window-inside-absolute-pixel-edges returns
the wrong y-offset when the per-buffer tab-line is enabled. The tab-line
is taken into account when computing the window's height, so we adjust
the y-offset after computing the height.

Fixed upstream in https://debbugs.gnu.org/cgi/bugreport.cgi?bug=75576.

* exwm-layout.el (exwm-layout--show): Adjust the window's y-offset to
include the tab-line.  (Bug #114)
@Stebalien
Copy link
Contributor Author

It looks like Eli wants to land the patch on master (Emacs 30 is getting real close) so I've configured the check to be Emacs 30 and below. Users compiling from Emacs master will have to wait for the patch to land upstream.

@minad
Copy link
Member

minad commented Jan 16, 2025

The issue is not that urgent. I think it is better to not delay the Emacs 30 release. For EXWM it shouldn't matter much if we check for version 30 or 31. Or are you aware of other packages or features which are broken by this? I am not sure if tab-line tabs are used widely. I've never used them in contrast to the tab-bar.

@Stebalien
Copy link
Contributor Author

I don't think this affects anyone else. I'm just waiting for a final call from the Emacs maintainers before merging this. And yeah, this absolutely should not delay the Emacs 30 release.

@Stebalien Stebalien merged commit 9cdfe95 into master Jan 16, 2025
@Stebalien Stebalien deleted the steb/fix-tab-line branch January 16, 2025 18:06
@Stebalien
Copy link
Contributor Author

The fix was pushed to Emacs master so I'm merging this as-is.

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.

2 participants