Bugzilla – Bug 8996
web UI for Alarms confusing; not clear that alarms are per-player
Last modified: 2008-08-28 17:10:28 UTC
The Alarm Clock web page is linked not from player prefs, but from the general web Extras menu (I'm using Classic; no time to test all the skins). When you view the alarm clock page, it says nothing about applying to a particular player. It uses general-sounding language like "This setting allows you to turn on/off all alarms at once. If set to Off, no alarm will go off." when it really means for that player. Furthermore, the pulldown at the top of the pane lists *global* settings pages, not *player settings* pages. The web UI should be more clear. Suggestions: - show the per-player pulldown instead of global pulldown - at least display $client->name() prominently. Perhaps offer a pulldown with a list of the current players (with the current player selected) to make it more obvious this is per-player - use more clear language, e.g. "This setting allows you to turn on/off all alarms at once. If set to Off, no alarm will go off for Living Room." (not the best language, but could be easily changed without impacting l10n strings - when saving per-player settings such as alarm settings, display not just SETUP_CHANGES_SAVED but SETUP_CHANGES_SAVED FOR $client->name()
This bug does not exist in the default skin. Clicking the alarm clock link under extras in the default skin opens the alarm clock settings in a new window, with the player tab highlighted and the current player name clearly displayed at the top of the page in a drop-down allowing you to change player. When you do this in classic, it simply appears in the normal left-hand frame, under the heading of Home/AB Tester in the top frame. Definitely a bug. Don't worry about patches - the per-player config issue is all fine. It's probably just the link under Extras that needs fixing.
fixed at change 22327