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

Add Run / Configure instructions to README #614

Merged
merged 1 commit into from
Jul 28, 2022

Conversation

Kache
Copy link
Contributor

@Kache Kache commented Jul 15, 2022

And fix whitespace

Fixes: #613

README.md Outdated
@@ -94,9 +95,17 @@ brew tap homebrew/cask-versions
brew install --cask background-music-pre
```

# Configure

Select **Background Music** for `Output` and `Input` under `System Preferences > Sound`.
Copy link
Owner

Choose a reason for hiding this comment

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

Background Music should automatically set your default output device to the Background Music device when it starts up. Is that not working for you? It's also supposed to change your default device back when you close Background Music.app.

You should only need your default output device (and system output, i.e. sound effects) set to the Background Music device, though, not your default input device.

What do you think about moving this to the troubleshooting section? Maybe like this:

Make sure you have **Background Music** selected for `Output` in `System Preferences > Sound`.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated PR to reflect this behavior. This is a minor thing -- I was just momentarily confused after install b/c I didn't know where the executable was added, if I had to add something to my $PATH, or if it installed as a driver thing and it'd already "activated". For example, some brew formulas print out "how to get it running" step of "add to your ~/.zshrc" after install, as well as in brew info.

README.md Show resolved Hide resolved
@Kache Kache changed the title Add #Configure instructions to README Add Run / Configure instructions to README Jul 27, 2022
@kyleneideck kyleneideck merged commit 8090f12 into kyleneideck:master Jul 28, 2022
@kyleneideck
Copy link
Owner

Looks great. Merged, thanks.

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.

Add instructions to setup "Launch on Login/Startup" to README
2 participants