Bug 8954 – Remove Default Volume feature from new alarm code

Bug 8954 - Remove Default Volume feature from new alarm code

Summary:

Remove Default Volume feature from new alarm code

Status:	RESOLVED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	Player UI
Version:	unspecified
Platform:	All All

Importance:	-- normal (vote)
Target Milestone:	---
Assigned To:	Unassigned bug - please assign me!

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2008-07-31 06:12 UTC by Max Spicer
Modified:	2009-09-08 09:21 UTC (History)
CC List:	1 user (show)

See Also:
Category:	---

Attachments
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 Max Spicer 2008-07-31 06:12:45 UTC

Should the Default Volume setting be removed from the new alarm code, along with the corresponding Use Default Volume setting for each alarm?  Apparently some discussion has taken place about this and Dean is in favour.  Michael thinks otherwise, however, and requested I file this bug for public discussion.

For what it's worth, I do find it extremely useful to be able to make changes to the volume used by all my alarms in one go.

This would need to be decided soon as it is a feature change!

Comment 1 Michael Herger 2008-07-31 07:00:48 UTC

I'd rather have _one_ single volume setting than no default setting. I really won't define the monday's alarm louder than thursdays. My night live might be too boring.

Comment 2 Blackketter Dean 2008-07-31 07:24:20 UTC

My impression was that it cluttered the interface (I like removing things).

I agree with Michael, I think that one master alarm volume control is probably sufficient.

Comment 3 Max Spicer 2008-07-31 07:27:50 UTC

That would suit me fine.  Per-alarm volume was a feature of the old alarm clock, but I certainly never used it.  I'll make this change, but I can't guarantee to have it done until the weekend.

Comment 4 Max Spicer 2008-07-31 09:41:36 UTC

Removed per-alarm volume settings in change 22269.  I've made the necessary web changes but not touched Jive.

I had to add new strings for this.  ALARM_VOLUME and ALARM_VOLUME desc.  I've also removed the previous alarm volume-related strings.

Comment 5 Max Spicer 2008-07-31 13:34:29 UTC

Change 22282 updates the pref migration code so per-alarm volume settings from old alarms are now ignored.