View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0007820 | MMW v4 | Other | public | 2011-05-16 05:39 | 2011-05-25 21:18 |
| Reporter | rusty | Assigned To | |||
| Priority | urgent | Severity | tweak | Reproducibility | always |
| Status | feedback | Resolution | open | ||
| Product Version | 4.0 | ||||
| Target Version | 4.1 | ||||
| Summary | 0007820: UPnP: Display server status via icon | ||||
| Description | It would be really useful if the Media Server's status was displayed in the tree (by having a slightly different icon depending on the state of the server). Proposed states: Off: grey On: green light We could add other states (e.g. On and active [green flashing]), but the above would give the most useful feedback. Set to 'urgent' if this is trivial, because I'd like to get the icons in the initial skins (it'll be harder to change post-release). | ||||
| Tags | No tags attached. | ||||
| Fixed in build | |||||
|
|
It is not so trivial, because currently we show only available Media Servers, this involves introducing a new DB table for storing metadata about servers that were at least once available. This would involve some other issues, for example ability to remove them so that they would not mess the Media Servers node. |
|
|
Isn't it a question of just turning the light 'on' if there's at least one server with a status of 'running' ? And turning it 'off' if there are no servers with a status of 'running'? |
|
|
No, because now all servers available under the 'Media Servers' node have status 'running'. We don't store a metadata about unavailable servers. Or do you mean that we should add that icon just for the 'Media Servers' root node? |
|
|
Yes--I just mean having two states for the 'Media Servers' root node. In the future we could make this more advanced, but I think that this would be sufficient for 80% of cases. |
|
|
I am still not sure that this is a good idea: 1. This info would be rather meaningless, because for example in Windows 7 there is always at least one server running (I mean the WMP's server) although it doesn't share anything by default. 2. Current workflow is that UPnP doesn't need to be initialized at all until the user expands the Media Servers node or until the user defines an own media server. By implementing your proposal MM would need to check availability of media servers every 20 seconds just in order to display right icon (green/gray) although this info is rather meaningless as stated in 1). |
|
|
I agree with Ludek, let's at least defer until 4.1, so that we have more feedback re. UPnP. However, even then it probably isn't necessary to implement this. |
|
|
To clarify the intent of the feature: At the moment, when the user enables the UPnP server via the Options panel, there's no indication that it is actually working. Even after the user clicks 'OK' in the options panel, there's still no indication that it's working. The only way for the user to see that it's working is to return to the Options panel, after having clicked OK, and verifying that the service is 'running'. The intent of the feature was to get rid of that requirement by changing the state of the Server Sharing icon so that it appears 'on' whenever a _mediamonkey_ server is running (i.e. WMP server is irrelevant). Granted, this isn't perfect, because the 'on' state is generic to all servers (i.e. it's on even if only a single server is running), but that would solve 80% of requirements. In the future we could e.g. improve this by displaying a different icon in the tree for each server instance, but for now, this would solve the lack of feedback in most cases. Given the above clarification, is issue 2) really an issue? |
|
|
Ok, this could be implemented. |
|
|
I still don't understand. Because if user defines an own media server than according to 0007165 he will never see the server in MM interface. So highlighting the 'Media Servers' node is off-topic, because the node isn't showing the user defined servers, but the remote servers from others UPnP/DLNA apps and clients. |
|
|
Further discussed with Ludek: He's right that the icon probably doesn't help that much, since the server isn't listed there. That said, I understand Rusty's point - it would be nice to show the status somehow. Some other ideas: a. Show some relevant tooltips on the tree node, like 'Local UPnP server - running'. b. Show the local server in this node. As Ludek pointed out, other applications usually don't do it, on the other hand, it might be useful, because we would show that the server is running and also user could review what's actually being shared, etc. It could be optional, but I'd rather avoid adding new options. In any case, it doesn't seem to be really urgent, so we might defer this, see some 4.0 feeback and decide then. |
