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

fix: frame title still shown #108

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

azzamsa
Copy link
Contributor

@azzamsa azzamsa commented Dec 5, 2021

Hi.

(setq frame-title-format nil) and (setq frame-title-format "") doesn't work in my end. However setting it to (setq frame-title-format " ") works.

I think it because "" also evaluates to nil

@azzamsa
Copy link
Contributor Author

azzamsa commented Dec 8, 2021

I think we need to set icon-title-format too. As described here:

@rougier
Copy link
Owner

rougier commented Dec 15, 2021

Oh, good to know. What is you system (on OSX, I use a patched Emacs to get rid of the title entirely). Does yu PR make the title completely disappear?

@azzamsa
Copy link
Contributor Author

azzamsa commented Dec 15, 2021

Yes. It my previous version of Emacs. But Now I need to add one more additional line

;; No hostname in frame title
;; Without setting the `icon-title-format`. The window title will revert
;; back to its original value after loosing its focus.
(setq frame-title-format '("" invocation-name " - " "%b"))
(setq icon-title-format '("" invocation-name " - " "%b"))
❯ sfetch
OS: Debian GNU/Linux 11 (bullseye)
Emacs version: GNU Emacs 29.0.50 (build 3, x86_64-pc-linux-gnu, GTK+ Version 3.24.24, cairo version 1.16.0) of 2021-12-07
``

@rougier
Copy link
Owner

rougier commented Dec 15, 2021

Oh, Emacs 29 might be a bit too ahead of current (27.2) version. Does you "hack" works on 27.2 as well ?

@azzamsa
Copy link
Contributor Author

azzamsa commented Dec 17, 2021

Sorry. I only have GNU Emacs 29.0.50 on my end.

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