Skip to content

Option to Hide main window#98

Closed
diogovalada wants to merge 3 commits intosavbell:mainfrom
diogovalada:hide-main-window
Closed

Option to Hide main window#98
diogovalada wants to merge 3 commits intosavbell:mainfrom
diogovalada:hide-main-window

Conversation

@diogovalada
Copy link

I think there's not much use in showing the main window every time, and not start listening right away. Even more so when the settings are always easily accessible via the tray icon.

Having the option to hide the main window enables, for example, running WhisperWriter at OS startup and start listening immediately to the hotkey, instead of compulsorily seeing the window every time.

@JdotCarver
Copy link

JdotCarver commented Feb 17, 2025

I've implemented #82 on my end and I :

  • Never see the startup window.
  • Have it launch on OS Startup.

How is this PR different ?

BTW : This PR also changes the default sample rate.

@diogovalada
Copy link
Author

I've implemented #82 on my end and I :

  • Never see the startup window.
  • Have it launch on OS Startup.

How is this PR different ?

BTW : This PR also changes the default sample rate.

No, it's the same, you're right, I had not seen there was already a PR implementing this.

BTW : This PR also changes the default sample rate.

It does not, if you look at "Files Changed"

src/config.yaml Outdated
min_duration: 100
recording_mode: press_to_toggle
sample_rate: 32000
sample_rate: 16000

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I might be a GitHub noob, but this looks like a sample rate change, isn't it?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, it reverts it back to 16k?

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, it's only the commit. The overall diff doesn't change it.

@Enyium
Copy link

Enyium commented Feb 19, 2025

I don't know why this would need to be an option at all. When you need the settings, you can get to them via the tray icon.

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