Xbmc nav sound volume setting #475

Closed
wants to merge 2 commits into
from

Projects

None yet

2 participants

@energy6
Contributor
energy6 commented Oct 7, 2011

Added setting to control navigation sound volume relative to master volume in percent.
The nav sounds are overwhelm with respect to video/music. The new setting allows the user to set the nav sound volume with respect to the global volume.

@jmarshallnz
Member

Sounds like we need to set a better default, rather than have yet another setting. What volume level sounds most suitable to you?

@energy6
Contributor
energy6 commented Oct 9, 2011

I have set the nav sounds to 20 percent of master volume. Typically this leads to about 5 to 10 percent absolute volume.
What do you mean with "set a better default"? How do you want to change the nav sound volume? Or do you want to fix the nav sound volume at the default level?

@jmarshallnz
Member

Fix the nav sound volume at a defualt level.

@jmarshallnz
Member

Please re-do using a default value for percentage of actual volume so as to get rid of the UI setting.

@nemphys nemphys pushed a commit to nemphys/xbmc that referenced this pull request Mar 24, 2012
@opdenkamp opdenkamp pvr: CPVRChannelGroups::GetGroupAll() doesn't need a lock. the group …
…will always be present at position 0. locking order was wrong anyway. closes #475
34569ef
@margro margro pushed a commit to margro/xbmc that referenced this pull request Mar 24, 2012
@opdenkamp opdenkamp pvr: CPVRChannelGroups::GetGroupAll() doesn't need a lock. the group …
…will always be present at position 0. locking order was wrong anyway. closes #475
f794d2c
@energy6
Contributor
energy6 commented Jun 15, 2012

Done in 35824a0 by using an advancedsetting instead of a gui setting. New pull request follows.

@energy6 energy6 closed this Jun 15, 2012
@tru tru added a commit to plexinc/plex-home-theater-public that referenced this pull request Feb 22, 2013
@tru tru Fix PlexHTHelper stupidity.
Fixes #475
3510af6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment