Bug 12847 - Preset HOLD to save preset
: Preset HOLD to save preset
Status: CLOSED FIXED
Product: SB Radio
Classification: Unclassified
Component: Menus
: Include FW version in comment
: PC Other
: P1 normal (vote)
: 7.4.0
Assigned To: Ben Klaas
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-07-14 09:47 UTC by Wadzinski Tom
Modified: 2010-05-27 14:46 UTC (History)
3 users (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Wadzinski Tom 2009-07-14 09:47:03 UTC
Emulate the Boom behavior - press and hold a preset to set the current browse item or, if the NP screen is up, the current song.

Currently, only playing presets (which play first 6 favorites) is working on baby, but saving is not. This will require some SC changes as well.
Comment 1 Wadzinski Tom 2009-07-15 14:40:58 UTC
fixed in SC 27588
Comment 2 Wadzinski Tom 2009-07-15 14:59:25 UTC
forgot about the NP flavor of this
Comment 3 Ben Klaas 2009-07-15 20:44:11 UTC
there's more too...

XMLBrowse item support
UI for popup on add-to-favs

Tom, we can work on this together Thursday a.m.
Comment 4 Ben Klaas 2009-07-17 11:59:29 UTC
XMLBrowse item support added in SC, r27644
Comment 5 Ben Klaas 2009-07-17 12:37:09 UTC
UI for popup is not to design spec, but it's no longer hideously broken with r6650. Good enough for CAT/CXR.

FYI, showBriefly behavior, layout, etc. are going to be a fairly major effort between CXR and 7.4.
Comment 6 Weldon Matt 2009-08-19 15:01:28 UTC
another note (from CXR and I agree):

if the preset is a playable item, pressing the preset button should:

A - start playback of that item, and
B - launch the Now Playing screen
Comment 7 Adrian Smith 2009-08-23 04:10:20 UTC
There's a problem with baby presets - press and hold sets a different preset from the one it plays...  There's code in SC to offset the set param by 1, but I am not sure where it should change to only impact the baby buttons.
Comment 8 SVN Bot 2009-09-06 09:10:58 UTC
 == Auto-comment from SVN commit #7430 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=7430 ==

Bug: 12847, 13875
Description: add toast_popup_textarea style to deal with text-only showBrieflies
fix titlebar height in multiline_text_list for portrait skin
Comment 9 SVN Bot 2009-09-06 09:12:03 UTC
 == Auto-comment from SVN commit #28452 to the slim repo by bklaas ==
 == https://svn.slimdevices.com/slim?view=revision&revision=28452 ==

Bug: 12847
Description: showBriefly for set-preset
Comment 10 SVN Bot 2009-09-06 10:20:49 UTC
 == Auto-comment from SVN commit #28453 to the slim repo by bklaas ==
 == https://svn.slimdevices.com/slim?view=revision&revision=28453 ==

Bug: 12847
Description: Allow set-preset to work for NP item by sending playlist_index param
Comment 11 SVN Bot 2009-09-06 10:21:53 UTC
 == Auto-comment from SVN commit #7432 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=7432 ==

Bug: 12847
Description: add setPresetCurrentTrack service in slimbrowser
add actionListeners in NP for set_preset_ 1-6 to call setPresetCurrentTrack
requires SC r28453
Comment 12 SVN Bot 2009-09-06 10:26:23 UTC
 == Auto-comment from SVN commit #7433 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=7433 ==

Fixed Bug: 12847
Description: push to NP screen after hitting preset key
Comment 13 James Richardson 2009-10-05 14:36:11 UTC
This bug has been marked as fixed in the 7.4.0 release version of SqueezeBox Server!
    * SqueezeCenter: 28672
    * Squeezebox 2 and 3: 130
    * Transporter: 80
    * Receiver: 65
    * Boom: 50
    * Controller: 7790
    * Radio: 7790  

Please see the Release Notes for all the details: http://wiki.slimdevices.com/index.php/Release_Notes

If you haven't already, please download and install the new version from http://www.logitechsqueezebox.com/support/download-squeezebox-server.html

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.
Comment 14 Chris Owens 2010-05-27 14:46:17 UTC
These bugs have all been marked resolved and belong to a component which is being removed.  Therefore they have been moved to the most applicable of the new components.