Some UI decision to take here: do we want to stick with the 11 level volume control? Is a popup the way to go? Maybe optional (click the current volume level to get finer control)?

We don't need to maintain 11 steps if the number isn't shown. (11 is a good joke but it's too obscure without the digits.)

I'm good with a slider with more control.

C'mon guys! This control has taken me _hours_ to implement the maths calculating the 11 steps :-).

I'll try to come up with a demo patch which will do this, so we can see what artwork we'll need to fit this into our layout.

*** Bug 8614 has been marked as a duplicate of this bug. ***

Can the volume level number be shown as a tooltip and on mouseover of the the slider control?

Another suggestion if the volume slider can be implemented: A mute button would be nice, where pressing the mute button a second time restores the previous volume level.

The mute request is covered by bug 562. Brian/Dean - should we consider this to be included with this change? We'd need some layout/artwork change too (at least a button icon).

Will consider post-7.2

FYI, in Touch/Nokia770 there are + - buttons to the right of the volume bars on the Now Playing page that allow you to move the volume up/down by increments of 1. You might give that a look see...

Thanks Ben.  That will be great.

The main complaint I have about the "few" volume steps is that the volume change is very abrupt – it really lets you jump outta the chair if turning too loud :-)

Would there be any chance of somehow "sliding" smoothly to the new volume setting, maybe over .5 or 1 s?

I want more granular control, too.

egander has a great idea, "If it's not too late to make a suggestion, how about having the "speakers" at either end of the volume bar increment by +/- 1, and leave the bar gradients at 10. I'm hoping that might be a really quick fix / implementation." http://forums.slimdevices.com/showthread.php?t=63895

Good idea egander!  Thanks for posting that yeswork.

Here's a mod that fixes this for me:

Edit \Program Files\SqueezeCenter\server\HTML\EN\html\SqueezeJS\Base.js

at line 426 (of my version - a 7.3.3 nightly), is the definition for SetVolume:

		setVolume : function(amount, d){
			amount *= 10;
			if (d)
				amount = d + amount;
			this.playerControl(['mixer', 'volume', amount]);
		},


change it to this:

		setVolume : function(amount, d){
			if (d)
				amount = d + amount;
			else
				amount *= 10;
			this.playerControl(['mixer', 'volume', amount]);
		},


I might change it to this though - as ten steps might be too small - this makes it 5 steps intead:

		setVolume : function(amount, d){
			if (d)
				amount = d + (amount * 2);
			else
				amount *= 10;
			this.playerControl(['mixer', 'volume', amount]);
		},

yeah, 5 steps is the ticket:

		setVolume : function(amount, d){
			if (d)
				amount = d + (amount * 2);
			else
				amount *= 10;
			this.playerControl(['mixer', 'volume', amount]);
		},

I should note that I have not tested it with the controller - just the default web UI.

*** Bug 14645 has been marked as a duplicate of this bug. ***

(In reply to comment #15)
> *** Bug 14645 has been marked as a duplicate of this bug. ***

Will this work with the controller?

(In reply to comment #15)
> *** Bug 14645 has been marked as a duplicate of this bug. ***

Will this work with the controller?

The default location of the file to edit has moved in 7.4.1.  It is now at:
F:\Program Files\Squeezebox\server\HTML\EN\html\SqueezeJS\Base.js

The setVolume function can be modified as before.

The mod has no effect on the controller.  The controller volume already steps one unit at a time (rather than 10 units as the web UI does).

Matt Weldon doesn't work for us any more.

(In reply to comment #19)
> Matt Weldon doesn't work for us any more.

I had the same trouble but got it working.  I had to restart my server (WHS machine) and then purge the browser cache (force refresh even worked fine one time).  Not sure the server restart was needed, but it looks like the jscript will get stored by your browser (Firefox in my case).

(In reply to comment #12)
> Good idea egander!  Thanks for posting that yeswork.
> 
> Here's a mod that fixes this for me:
> 
> Edit \Program Files\SqueezeCenter\server\HTML\EN\html\SqueezeJS\Base.js
> 
> at line 426 (of my version - a 7.3.3 nightly), is the definition for SetVolume:
> 
>         setVolume : function(amount, d){
>             amount *= 10;
>             if (d)
>                 amount = d + amount;
>             this.playerControl(['mixer', 'volume', amount]);
>         },
> 
> 
> change it to this:
> 
>         setVolume : function(amount, d){
>             if (d)
>                 amount = d + amount;
>             else
>                 amount *= 10;
>             this.playerControl(['mixer', 'volume', amount]);
>         },
> 
> 
> I might change it to this though - as ten steps might be too small - this makes
> it 5 steps intead:
> 
>         setVolume : function(amount, d){
>             if (d)
>                 amount = d + (amount * 2);
>             else
>                 amount *= 10;
>             this.playerControl(['mixer', 'volume', amount]);
>         },


Personally, I prefer the single step increments.  It's all about having more detailed granular control.  Nonetheless, thanks so much for your effort Sean.  Greatly appreciated.

 == Auto-comment from SVN commit #30802 to the slim repo by mherger ==
 == http://svn.slimdevices.com/slim?view=revision&revision=30802 ==

Fixed Bug: 8763
Description: allow finer volume control in the default web UI. Clicking the +/- buttons will use 50 increments, clicking the bar 11.

Michael, can this enhancement be safely backported to 7.5?

Since a 7.5.2 release looks eminent I'll ask again: Can this be backported?

 == Auto-comment from SVN commit #31568 to the slim repo by mherger ==
 == http://svn.slimdevices.com/slim?view=revision&revision=31568 ==

Bug: 8763
Description: allow for more granular volume control