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

Setup for custom title delay settings. #280

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

clementpadovani
Copy link

Added the possibility to change the delay for the app to set a tab’s title.

This can be achieved by running the following command:

defaults write com.googlecode.iterm2 DelayedTitleCustomDelayTimeInterval -float 0.5

If no value is set in the defaults then the default value of 0.1 is used.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.002%) to 32.335% when pulling 494d037 on ClementPadovani:feature/tabTitleCustomDelay into 29b75f7 on gnachman:master.

@gnachman
Copy link
Owner

gnachman commented May 3, 2016

Out of curiosity why would you want to do this?

@clementpadovani
Copy link
Author

clementpadovani commented May 3, 2016

Hey @gnachman,

I missed the instant process name change after migrating from Terminal to iTerm.
And other might want a lower/higher delay time.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 32.344% when pulling 2aeb9a5 on ClementPadovani:feature/tabTitleCustomDelay into 4e1f5fa on gnachman:master.

@gnachman
Copy link
Owner

gnachman commented May 3, 2016

I'd be surprised if a .1 second delay were perceptible. The lion's share of the delay is that Terminal does some undocumented magic to get the process name instantly, whereas we have to poll periodically every 1/2 second. If you really want to speed it up increase the polling interval in ProcessCache.m line 353, but it'll cost you some CPU.

@clementpadovani
Copy link
Author

Alright, thanks @gnachman for your help. I’ll try messing around with ProcessCache when I have more time.

TBH it’s really a detail and it’s some of my classmates noticed the difference when switching from the stock term to iTerm that’s all.

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.

3 participants