Bug 8976 – non-default web ui for alarm shows blank new alarm

Bug 8976 - non-default web ui for alarm shows blank new alarm

Summary:

non-default web ui for alarm shows blank new alarm

Status:	CLOSED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	Web Interface
Version:	7.2
Platform:	PC Other

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

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2008-07-31 19:36 UTC by KDF
Modified:	2009-07-31 10:26 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 KDF 2008-07-31 19:36:54 UTC

field for time is set up by the Settings.Alarm.initTimeControls, which is not loaded by EN-based skins.  the js in question is located in Default skin only.

Comment 1 KDF 2008-07-31 19:47:45 UTC

sorry.  I misread the page initially.  The problem is that the new alarm shows up with blank settings.  ideally this would not be there until the user clicks a 'new alarm' button.

Comment 2 Michael Herger 2008-08-02 05:13:04 UTC

You're right, all the fancy stuff is Default only. But it's basically working, just not nice, isn't it?

I'll give it some love later next week.

Comment 3 KDF 2008-08-02 21:18:05 UTC

works great, just caught me off guard.  In fact, much more trivial now that I realise the real problem. If hiding is too fancy for en-based skins, even moving the 'new alarm' to the bottom might help.

Comment 4 Michael Herger 2008-08-04 08:32:08 UTC

change 22351 - add the show/hide JS magic to useAJAX enabled skins

Comment 5 James Richardson 2008-08-26 14:09:17 UTC

Verified fixed in
SqueezeCenter 7.2-22900

Comment 6 James Richardson 2008-12-15 12:34:37 UTC

This bug has been fixed in the 7.3.0 release version of SqueezeCenter!

Please download the new version from http://www.slimdevices.com/su_downloads.html if you haven't already.  

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.

Comment 7 Chris Owens 2009-07-31 10:26:19 UTC

Reduce number of active targets for SC