View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0010746 | MMW v4 | Synchronization | public | 2013-04-17 22:20 | 2013-05-06 19:45 |
| Reporter | peke | Assigned To | |||
| Priority | immediate | Severity | feature | Reproducibility | N/A |
| Status | closed | Resolution | fixed | ||
| Product Version | 4.1 | ||||
| Target Version | 4.1 | Fixed in Version | 4.1 | ||
| Summary | 0010746: Playlists: Ability to force output Playlist format | ||||
| Description | Currently Even Device only support PLS playlist if set MMW will export M3U is selected In playlist settings. There should be a way to select type of exported playlist with Dropdown For compatibility with older device profiles M3U should be default unless it is explicitly set that device only support PLS playlists. Most Devices handle that internally, but problem may arise where user uses removable media (SD Cards, Flash Drives, External Hard Drives) directly in MMW and not connected thru Device that will play those media files and device support only specific Playlist formats. Simple one way export of those playlists on Sync should be easy to implement. UI shouldn't change too Much "[x] Sync Playlist___________Playlist Format: [ M3U |\/]" Playlist formats: http://gonze.com/playlists/playlist-format-survey.html | ||||
| Tags | No tags attached. | ||||
| Fixed in build | 1636 | ||||
| related to | 0010757 | closed | Ludek | M3U is still synced despite the user configured only PLS as supported playlist format |
| parent of | 0011634 | closed | Ludek | Playlist supported format setup |
| has duplicate | 0010772 | resolved | rusty | Sync Playlists: UI Changes |
| related to | 0010779 | closed | Ludek | Add support for generation other playlist formats |
|
|
Wouldn't be this easy to add? |
|
|
You are true that currently it is quite hidden via <Device> -> Options -> Auto-conversion -> Supported formats on device I agree that single combo box in <Device> -> Options -> Playlists would be better and much more intuitive. I would add the combo Target Format: [M3U^] just below [ ] Organize playlists using... and the four last checkboxes would be hidden if M3U is not selected (because they are M3U only related) Rusty, do you agree? |
|
|
Fixed in build 1633. |
|
|
Great. I'd just updated the wording/order as follows: . . . Destination directory: _\Playlists\_ [ ] Organize Playlists using: _prefix for each playlist type_^ Playlist format: ____^ . [x] Use unicode (UTF-8 encoding) for M3U playlists . [x] Use extended M3U . [ ] Force relative paths . [ ] Use Linux folder separator ('/' instead of '\') (note: the last 4 options should be greyed out or hidden if not relevant to the format chosen) |
|
|
Fixed in 1633. |
|
|
Verified 1633, but would like to consider two small tweaks. 1. I would prefer Gray out instead of Hide due the fact that maybe some playlist formats could support some additional settings? PLS for example uses URL/Linux Separator Will see what user say. 2a. I would rather put Playlist format: ____^ before Sync Playlists to ensure that it is clear and on Global level for this device profile 2b. I would rename "Sync Playlists" -> "Sync additional Playlists" to clear confusion from Selected Playlists or make "Sync Playlists" Global Switch so that Selected Playlists are not Synchronized at all but only tracks from selected Playlists (I would prefer just rename, due the possible regressions with existing profiles) |
|
|
Reminder sent to: Ludek Few bugs: 3. UI is not corrected to proposed format : Playlist format: ____^ . [x] Use unicode (UTF-8 encoding) for M3U playlists . [x] Use extended M3U . [ ] Force relative paths . [ ] Use Linux folder separator ('/' instead of '\') 4a. Playlist format should be selectable Even Sync Playlist is not selected unless MMW sync selected playlists according to first one supported? 4b. Playlist Format selection do not stick if Sync Playlists is not selected. |
|
|
I uploaded Video File showing bug 4a. onto FTP. |
|
|
Peke, if '[x] Sync Playlists' is disabled then playlists are not synced at all (tested both auto-sync and manual sync via "Send to -> <Device>"). Tested latest build (1633) and also verified in code. I guess this is all right that playlists are synced only when '[x] Sync Playlists' is enabled or if the 'Playlists' tab is missing completely (like in case of iPods/iPhones). So from my perspective the current UI (in build 1633) is OK. Note: Haven't found the Peke's video on FTP. |
|
|
2b. MMW always Sync Playlists to MMA Even Sync Playlists are disabled Haven't tested Touch, but there should be ability to not sync playlists even on iDevices where Playlists is missing. 4a. My bad, Video uploaded again NOTE: Other than these two I have created UI mockup and changes list into 0010772 to make UI more complete and friendlier. |
|
|
1) I think hiding the irrelevant options is fine. As long as we don't hide options that should be presented to the user. e.g. if PLS should have an option to disable the URL/Linux separator and it's currently missing, then it should be fixed. 2a) There are a number of different types of devices that MMW can sync to: - USB MS devices: Playlist file should always be generated on the device for selected playlists, and user should have the option of creating others. - MTP devices: 'virtual' Playlist should always be generated on the device for selected playlists, and user should have the option of creating others - Apple devices: No config should be required--except if the user is using custom firmware - Android devices (MTP / USB MS): See USB MS and MTP above for the different cases Given this, I'm think I understand your suggestion to make playlist configuration an option that's always active, since: - For USB MS devices, some configuration re. generation of playlist _files_ is always required. - For MTP devices, the config is probably not generally required except in the cases where: a) the virtual playlist format communicated by MTP for the device is incorrect (I believe there was a case of this) b) the device using the MTP driver is a USB MS device. Based on the above, Peke's suggestion would seem to make sense, though I would additionally get rid of the option to [x] Sync playlists, and modify the presentation to: Playlist format: ____^ . [x] Use unicode (UTF-8 encoding) for M3U playlists . [x] Use extended M3U . [ ] Force relative paths . [ ] Use Linux folder separator ('/' instead of '\') Destination directory: _\Playlists\_ Also generate playlist files on the device for selected: [ ] Artists [ ] Albums [ ] Locations [ ] Organize Playlists using: _prefix for each playlist type_^ (this option can be greyed out unless at least one of either [ ] Artists, [ ] Albums, or [ ] Locations are selected) 2b) Yes--it's strange to have an option to 'Sync playlists' if it has no effect. I think that it's safe to assume that if MMW syncs with a device that supports a database (i.e. MTP devices, Apple Devices, Android devices) that a playlist should always be created on the device for any playlists selected by the user. For this reason, the proposal above eliminates the '[x] Sync Playlists option' (always assuming that playlists should be synced if selected). 3) Are you referring to the fact that the options are listed in the wrong order? Minor issue, but sure it can be fixed. 4a) Makes sense that it should be selectable (see discussion in 2a). 4b) Moot point if we go ahead with the changes above 5) Currently, the only formats that are displayed as options are those that are chosen by the user in the 'Supported formats' section of the auto-conversion config. I would suggest that most users won't look at that config in detail. Is that config really required? Would it make sense to allow users to choose from any format in the Playlist config? |
|
|
2a. USB MS: I strongly do not agree that Playlists are always synchronized especially as most Car head units do not support playlists. In all other cases Rusty is correct Maybe to avoid that on supported devices we can add this UI that in case Nothing is selected MMW do not sync playlists making "Selected playlists" default (like I made Mockup in 0010772). That way will will have both options and retain Rusty proposed changes. Eg. if Nothing selected than no playlists are generated. Generate playlist files on the device for selected: [x] Synched Playlists [ ] Artists [ ] Albums [ ] Locations 3. Yes, I'm referring to that This way (0010772) it looks more clear and leave space for further improvements especially when common options like Relative paths will be on same place where available and hide options will be shown bellow. Based On above we can easily remove Sync Playlists Checkbox and handle it based on Selected options. |
|
|
I like the Peke's mockup http://www.ventismedia.com/mantis/file_download.php?file_id=1412&type=bug&show_inline=1&file_show_inline_token=2013042577d507e70779175b21877e6d5d2841293af07cae where '[x] Selected Library Playlists (recommended)' supplies the current '[x] Sync Playlists' checkbox. 5) This is not true, there are shown formats that device supports and that we can create/sync. e.g. If I connect my Galaxy Nexus via MTP then 'PLS, PLA, M3U' are listed in the combo regardless what is configured in Supported formats. |
|
|
2a) I'm beginning to appreciate the rationale for the design of the original (i.e. current) UI, as it supports: i) cases where the user needs to just sync virtual playlists (i.e. for MTP devices), but not worry about generating any particular file type. e.g. [x] Sync playlists is enabled, but other options are disabled ii) cases where the user needs to generate (or not generate as Peke suggests is required) playlist files That said, Ludek made a point that there isn't really any benefit to treating 'Virtual Playlists' on an MTP device vs Generated Playlists on a USB MS Device any differently, and assuming Peke is right about the fact that MM needs a mechanism to not sync Playlists, we can go with the approach suggested by Peke. 3) The rationale for the organization of the dialog as below is that the first section (formats/destination directory) applies to most devices, whereas the second (re. generated playlists) is relevant mainly to USB MS devices. HOWEVER, if we follow the suggestion in 2a, which suggests that Playlists should _not_ be created unless specifically configured via 'Generate device Playlists' settings, then all configuration should be sub to Generate device playlists (since the settings are irrelevant unless device playlists are being generated). 5) Per discussion with Ludek, the primary purpose of the 'Supported formats'>Playlists would be to determine what formats are displayed in the 'Playlists format:' configuration. BUT it doesn't make sense to have these two items configured in completely separate locations. To rectify this, it was proposed that either: a) We get rid of Playlists from the 'Supported formats' dialog, and just list all formats supported by MM in the list of playlist formats, highlighting those that the device indicates are supported. This would appear as follows: Playlist format:________^{M3U (supported), M3U8, PLA (supported), PLS, XSPF, ASX, WAX} The downside with this approach is that there's no way to include in the device profile a list of supported playlist formats for the device--only the preferred format. b) The 'Supported formats:' for playlists should appear on the Playlists dialog. e.g.: Playlist format:__M3U____^ [Set formats] If the user clicks [Set formats]--> i) 'Supported format configuration' dialog is shown with Content=Playlists' (i.e. it would open an existing Playlist entry, or create a new one automatically). OR ii) Compatible Media Formats is shown, with existing Playlist entry highlighted. If the user click 'Add supported format', then MM should launch the 'Supported format configuration' dialog with Content=Playlists. I lean towards option b), but either approach would seem to work. So based on all of the above, and assuming we go with 5b), the dialog would appear as follows--which is actually quite similar to the current dialog: [x] Sync playlists (recommended) Generate device playlists for: [ ] Artists [ ] Albums [ ] Locations Playlist format: _M3U_^ [Set formats] . [x] Use unicode (UTF-8 encoding) for M3U playlists . [x] Use extended M3U . [ ] Force relative paths . [ ] Use Linux folder separator ('/' instead of '\') Destination directory: _\Playlists\_ [ ] Organize Playlists using: _prefix for each playlist type_^ |
|
|
Implemented in build 1635 as suggested -- i.e. used the option 5b) The only that remains to add is support for generation of more playlist formats in USB MS mode, tracked as 0010779 |
|
|
Reopen: 6. Set format -> Content is not Locked to Playlists 7. Clicking either OK/CANCEL Open Compatible Media Dialog Instead of returning to Playlists @Rusty NOTE: I do not see much functionality improvements than we exposed features that were hidden behind auto Conversion window? eg. There is stil PLaylists listed In compatible Media Dialog. |
|
|
8. There is still no way to disable "Selected Sync Playlists" synching. Users with large library uses Auto Playlists to filter sync lists for easier selection of files that needs to be Synched onto device (eg. Library tracks added in past month) but like to have their playlists organized in Artists, Albums, Location without those Auto Playlists Mockup I proposed in 0010772 solves that issue while retaining functionality (which Ludek confirmed it) |
|
|
6. Do you mean that the Content combo should be disabled or hidden? 7. Yes, this was according to Rusty's proposal, but maybe we could skip the "Compatible Media Dialog" in this case. 6&7. I think that the goal wasn't to separate the playlists from "Compatible Media Formats" dialog, but rather have a shortcut there from the 'Playlists' tab so that users know that it is related. 8. I don't understand, I re-verified that if the main '[ ] Sync Playlists (recommended)' checkbox is disabled then no playlist is synced. i.e. it doesn't matter whether it is selected playlist on sync list or a playlist to be created via Send to -> <device>. Simply no playlist is synced. This also seems to be logical. Could you re-verify this to confirm my words. If you are saying that a playlist is synced even if the checkbox is disabled then please give me more detailed steps to reproduce. And please ensure that you clicked [Apply] button before syncing. |
|
|
Re. 6/7: The most significant problem previously was the fact that users didn't understand how to configure a playlist from within the Playlists config. Now, they can. If it's trivial to fix 7, then that would be useful, but it might also cause problems (e.g. in cases where > 1 Playlist entries exist--unless MM forces supported Playlists to always be configured within a single entry, which would probably be a good idea). Re. 8): I haven't tested it out, but I suppose that Peke's point is that currently the UI _looks_ as if Sync Playlists is superior to the option to generate device playlists for Albums, Artists... even though it isn't / shouldn't be (i.e. whether the user selects [ ] Album shouldn't be dependent on whether the user selects [ ] Sync playlists even though the current UI implies that it is). Fixing this is trivial--it should look like: [x] Sync playlists (recommended) Generate device playlists for: [ ] Artists [ ] Albums [ ] Locations Instead of [ ] Sync playlists (recommended) . . Generate device playlists for: . . [x] Artists . . [ ] Albums . . [ ] Locations Morever, it should be possible to select Artsts, Albums, or Locations, even if 'Sync Playlists' isn't selected. Note: to be clear, if Sync Playlists is not selected, then the content of selected playlists syncs--just the playlist itself doesn't sync to the device. |
|
|
6. Content combo should be disabled as it is just shortcut. 7. It should skip "Compatible Media Formats" dialog as it is not Parent as shortcut directly goes to Playlists Content dialog and on selection it should go back to Playlists settings which is parent window. 8. You are right Rusty, but I was not referring to that. If user for example uses only Playlists as criteria for sync (Auto-Playlists like I put as example in 0010746:0035829) but do not want those playlists to be synched and only Album, Artists should be created on Sync. Currently This setting is not Possible. [ ] Sync playlists (recommended) Generate device playlists for: [x] Artists [x] Albums [ ] Locations |
|
|
All (6,7,8) is fixed in build 1636 + MM forces supported Playlists to always be configured within a single entry (as suggested by Rusty) |
|
|
Verified 1636 asked feedback from Rusty to confirm. Further improvements will be handled in new bugs accordingly when needed. |
|
|
Verified--UI looks good. Peke tested all functional aspects. |
