Bug 6138 - Transporter digital input mode plugin does not appear.
: Transporter digital input mode plugin does not appear.
Status: CLOSED FIXED
Product: SB Controller
Classification: Unclassified
Component: Browser
: unspecified
: Macintosh Other
: P5 normal (vote)
: 7.0
Assigned To: Felix Mueller
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2007-11-12 07:20 UTC by Blackketter Dean
Modified: 2008-05-15 13:00 UTC (History)
1 user (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Blackketter Dean 2007-11-12 07:20:59 UTC
 
Comment 1 Ben Klaas 2007-12-16 16:12:37 UTC
Dean, would you mind adding this to where it should go in the settings UI wiki?

http://wiki.slimdevices.com/index.cgi?JiveSettings

(just an FYI, everything in settings is now rearranged as specified in the wiki in the mainMenuRework branch; you will see the change when the merge happens later this week)
Comment 2 Blackketter Dean 2007-12-16 16:52:23 UTC
I think it should go into Extras for now, able to be promoted to Home in the Home menu setting....
Comment 3 Blackketter Dean 2007-12-16 16:56:42 UTC
Hm, I already had it in the top level home menu (as it currently is on the transporter UI:

http://wiki.slimdevices.com/index.cgi?UserInterfaceHierarchy

That seems reasonable, since it will be enabled by default there and easy to find.

Users can turn it off via the Home Menu setting.
Comment 4 Ben Klaas 2007-12-16 21:10:29 UTC
huh, that's odd...it's a setting, isn't it? I'll take your word for it that it should be someplace other than "settings" :)

Comment 5 Blackketter Dean 2007-12-17 06:10:24 UTC
Ah, no.  It's more like content.  You "Play" from one of the inputs.  It's top level now, let's preserve that and consider also having it in extras.
Do you have a Transporter?
Comment 6 Ben Klaas 2007-12-17 06:45:01 UTC
>Do you have a Transporter?

I do. It's pretty, black, and has this big knob in the middle. Pretty sweet.

Seriously, I'm working on this in the mainMenuRework branch. Hopefully I can get it in this morning.
Comment 7 Ben Klaas 2007-12-17 22:00:47 UTC
Is there a doc you could point me to that describes what digital inputs are and how they should be used? Not really grokking this...

I have the barebones done, but since I don't know the expected behavior I don't know how to test...
Comment 8 Ben Klaas 2007-12-17 22:01:24 UTC
Also, can these be set via the web UI? Is that what "Clock Source" is?
Comment 9 Blackketter Dean 2007-12-18 06:14:17 UTC
The basic idea is that there are four choices for input, corresponding to the four connectors on the back:

Balanced AES-EBU
Optical Toslink
S/PDIF RCA
S/PDF BNC

Pressing PLAY on any one of these will put a fake URL on the playlist which triggers the Transporter to "play" the input to the output of the DAC.  

Note: Since there's no song info on them, pressing the center button should also trigger a play and should appear with an play icon and not a >. See bug 6220.

This menu already exists in both the web UI and the player UI home menu when connected to SC, SN and even in the disconnected player setup area, so we're not really breaking any ground here.
Comment 10 Ben Klaas 2007-12-26 08:25:49 UTC
I'll leave this for 7.0, but it's priority P5

Comment 11 Blackketter Dean 2008-01-18 10:33:49 UTC
Felix, can you help with this?
Comment 12 Felix Mueller 2008-01-22 09:07:03 UTC
Should the Digital Inputs screen have an icon in the upper right corner? If yes, where can I find it?
Comment 13 Ben Klaas 2008-01-22 09:11:22 UTC
you'll want to give the window a titleStyle of 'settings', something like this:
window         => { titleStyle => 'settings' },

Felix, are you putting this into Slim::Control::Jive.pm? That's where the rest of the player settings get sent, so that's where it should probably go. 
Comment 14 Felix Mueller 2008-01-22 09:34:25 UTC
Ok, got the icon working. I am not sure though if a 'setting' icon is the right thing to do?

No, I have everything in the Digital Inputs plugin (Plugin.pm) right now. Richard felt it should be in the plugin and after having everything in there I also think it's the right place.

The only thing I am not sure is, how to get properly notified if the user changes player. I need to be able to add/remove the Digital Input Item in the home menu accordingly.

Currently I subscribe to 'menustatus' which works fine, but maybe there is a better way to do this.
Comment 15 Ben Klaas 2008-01-22 10:47:28 UTC
Felix, there's also a 'playerstatus' subscription that's made. Not sure which one is more appropriate-- both get unsubscribed when leaving a player and subscribed when adding a player.

Since this is a menu item, menustatus seems correct.
Comment 16 Felix Mueller 2008-01-24 07:34:38 UTC
Added Digital Inputs menu on SBC home menu if controlling a TR
Comment 17 James Richardson 2008-05-15 13:00:14 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