Bug 7047 – web volume indicator should show closest volume step

Bug 7047 - web volume indicator should show closest volume step

Summary:

web volume indicator should show closest volume step

Status:	CLOSED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	Web Interface
Version:	unspecified
Platform:	All All

Importance:	-- minor (vote)
Target Milestone:	7.x
Assigned To:	Michael Herger

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2008-02-08 15:03 UTC by Peter Watkins
Modified:	2009-09-08 09:18 UTC (History)
CC List:	0 users

See Also:
Category:	---

Attachments
implements logic described in opening comment (1.38 KB, patch) 2008-02-08 15:04 UTC, Peter Watkins	Details \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Peter Watkins 2008-02-08 15:03:28 UTC

I use the Handheld skin often to control my systems. It has the usual 1 through 11 indicators for current volume. If the current volume is a multiple of 10 exactly, the Handheld skin will show the volume. If it's not an exact multiple of 10, there's no indication what the player's volume is. My patch makes SC7 "round down", e.g. volume "2" will be used not just for volume "10", but also 11 through 19. This allows the Handheld skin to show the current volume level, and works fine in the New Default and Classic skins, too.

Comment 1 Peter Watkins 2008-02-08 15:04:04 UTC

Created attachment 2832 [details]
implements logic described in opening comment

Comment 2 Blackketter Dean 2008-02-08 15:35:26 UTC

Does this affect all skins or just Handheld?  If it affects the default skin, it's a 7.0 bug, otherwise, let's punt to 7.0.1

Comment 3 Peter Watkins 2008-02-08 18:48:02 UTC

It definitely does not affect Default or Classic. As far as I know, only Handheld is affected. I think this bug is a carryover from SS 6.x, at that.

Comment 4 Blackketter Dean 2008-02-10 21:04:55 UTC

We'll take a look at this post 7.0.

Comment 5 Michael Herger 2008-02-11 00:10:08 UTC

Only EN and Handheld are using that template. Thus any other skin is not touched. Easy enough to fix for 7.0.1

Comment 6 Michael Herger 2008-02-13 02:18:15 UTC

change 17493 - thanks!

Comment 7 James Richardson 2008-05-08 13:28:06 UTC

(In reply to comment #6)
> change 17493 - thanks!
> 

Verified fixed in 7.0.1 - 19325 

Peter, if you see any problems with the change, please reopen the bug

Comment 8 James Richardson 2008-05-15 12:27:43 UTC

This bug has recently been fixed in the latest release of SqueezeCenter 7.0.1

Please try that version, if you still see the error, then reopen this bug.

To download this version, please navigate to: http://www.slimdevices.com/su_downloads.html

Comment 9 Chris Owens 2009-07-31 10:16:51 UTC

Reduce number of active targets for SC