View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0008705 | MMA | Synchronization | public | 2011-11-15 13:27 | 2013-04-28 20:00 |
| Reporter | jiri | Assigned To | |||
| Priority | urgent | Severity | minor | Reproducibility | N/A |
| Status | closed | Resolution | fixed | ||
| Product Version | 1.0.1 | ||||
| Target Version | 1.0.1 | Fixed in Version | 1.0.1 | ||
| Summary | 0008705: Synchronization UI | ||||
| Description | 1. Options screen has the following items: a. Pair Device - lists all accessible MM servers (i.e. UPnP devices) and lets user to select the one to be used (note that we should use an unique ID, not an IP address which can change anytime later) b. Synchronization list - shows a list of items for synchronization as specified in Mantis 0008701. In case a device isn’t configured yet or isn’t accessible, the same as in 1a. is shown. 2. The Home screen (to be created - 0008704) has ‘Synchronize’ button that starts synchronization. In case no device is connected or configured, 1a. (see above) is shown. In case the selected device hasn’t been configured yet, 1b. is shown. 3. When synchronization starts, there’s: a. Synchronization screen shown, where user can: i. See progress (track count already synced and to be synced, currently downloaded track, etc.). ii. Cancel synchronization using a button. iii. It isn’t necessary to stay on this screen, synchronization is done on background. b. Status bar icon is added. User can click it, which shows the 3a. screen. | ||||
| Tags | No tags attached. | ||||
| Fixed in build | 93 | ||||
| related to | 0008631 | closed | marek | MMA | 'Select MediaMonkey server' options item is not needed |
| related to | 0009013 | closed | Ludek | MMW v4 | Option to use media server as sync server |
| related to | 0008808 | closed | Ludek | MMW v4 | Improved bi-di sync implementation |
| related to | 0010034 | closed | rusty | MMA | Status screen improvements |
| related to | 0010036 | closed | martin | MMA | Clicking 'Close' on sync status screen should exit sync workflow |
| related to | 0009501 | closed | marek | MMA | Implement wi-fi sync confirmations for deletion of tracks |
| related to | 0010197 | closed | marek | MMA | Connection message is redundant |
| related to | 0010198 | closed | marek | MMA | Text improvements to the sync UI |
| related to | 0010257 | closed | marek | MMA | Facilitate installation of MMW server if not installed |
|
|
Added Activity with synchronization details. Added more complex synchronization management service. |
|
|
Apart from minor usability issues as discussed over IM, some UI changes should be made: 4. When synchronization finishes, the sync screen should change to something like 'Synchronization successfully finished', progress bar and Cancel button would disappear. This screen would automatically disappear in ~3 seconds. 5. In case sync is canceled, the same as in 4. would happen, only the text would be 'Synchronization cancelled'. |
|
|
Activity has been fixed. |
|
|
Upon initiating a first-time sync via 'Sync Now' in MMA, user is prompted to select a Sync Server, but then is given no feedback as to what to do next, which often will result in: a) failed sync b) sync of no content To improve usability, the workflow should be modified so that it is obvious what actions are required of the user at each step, and the UI should give clear feedback re. pairing status (unpaired / searching / paired with XX). For example: 0) When the user clicks 'Sync Now': - For the first time: MM should open the Pair device dialog (implemented) - On subsequent occasions: the Sync dialog should indicate in the heading what server is being synced 1) Once the user chooses the sync server for the first time (i.e. if it's a sync server that hasn't previously been configured), it should offer feedback re. what's happening, preferably without opening a new dialog (see how Jellybean does this for bluetooth). e.g.: instead of "Waiting for response...' "Waiting for the remote server... [Cancel]" "Waiting for the remote server... If this is your first time connecting to it, you'll need to accept the connection on the server in order to proceed. [Cancel]" 2) Upon successful pairing, the Sync options should remain showing (i.e. a Sync operation should not be initiated unless sync settings have been configured), and it should be obvious what to do next. e.g.: a) Once the pairing operation succeeds, the 'Pair Device; Select sync server' option should change to 'Paired with: <Server> (note the wording change for both) b) Change text: Set Internal Storage settings: <path> --> Sync settings (and change the state from greyed out to active once pairing is complete) Then display a line for each internal destination e.g. Nexus 7 Internal storage . . . . [Settings icon button] Nexus 7 SD card . . . . . . . . . [Settings icon button] c) It might even be a good idea to automatically open the Sync Settings dialog upon successful pairing (if there isn't more than a single destination on the device) so that it's obvious that users should then configure the sync settings. And if there are 2 devices, to automatically prompt the user to configure the external device. Edit: The process should be automated because otherwise the following results: User clicks 'Sync now' --> 'Choose server dialog appears' (and MMW prompts user whether to grant access) ---> Nothing happens because sync settings aren't configured. d) Within the Sync Settings dialog, add a Sync Now button (so that users who've gotten to this point after clicking the 'Sync Now' button in the main UI are able to proceed with the operation) Here's how this would look overall: ----------------- WI-FI Synchronization . Select sync server . Choose the MediaMonkey server to sync with / (if configured show:) Sync server: <Server> . Sync settings (greyed out if device isn't paired) ---------------------------------------------------- . <Device Name> <FriendlyPath1> . . . . [Settings icon button] . <Device Name> <FriendlyPath2> . . . . [Settings icon button] ----------------------------------------------------------------------------------------------- Sync settings: <DeviceName><FriendlyPath1> (note the change in Title, giving clearer context) --------------------------------------------------------------------- You can configure basic sync settings on your device, but for more detailed configuration, connect your device directly to the MediaMonkey server, and configure the Sync settings on the server directly. Sync list Browse remote server and choose media to sync [ ] Bidirectional sync Syncs tracks from this device to the server Delete tracks from the device? [ ] Delete only files from the sync list that aren't selected [ ] Delete all media files besides those selected in the sync list [Sync Now] |
|
|
As discussed over e-mail, the idea of greying out the 'sync settings' is problematic because in cases where MMA has been granted access to MMW and then MMW subsequently revoked access, sync settings entries for servers must be clickable in order to re-enable access. Consequently, the following change was suggested: ----------------- WI-FI SYNC . Select sync server . Choose a new MediaMonkey server to sync with / (if configured show:) Sync server: <Server> . . Sync settings ---------------------------------------------------- . . <Device Name> <FriendlyPath1> . . . . [Settings icon button] . . <Device Name> <FriendlyPath2> . . . . [Settings icon button] |
|
|
2) a)fixed in build 53 b)fixed in build 53 d)fixed in build 53 - only options screen |
|
|
2b) These lines are too thin now, quite hard to accurately press them by a finger. 2d) I don't see the Sync button anywhere in Options. :-? |
|
|
3) There was also an offline discussion about update of Sync progress UI. Based on IM discussion with Marek, I think that the following makes the most sense: Internal Storage ---------------- Uploaded 5 tracks // Finished parts are left on screen Downloading.... // Parts in progress are being updated SD Card ---------------- to be processed All the statuses that can happen for a particular storage are: (T) = Temporary - when finished, will be removed from screen (K) = Keep - when finished, completion status will remain on screen 0. to be processed // (T) Sync of this storage hasn't started yet. 1. Analyzing (T) 2. Uploading track x of y ... (K) // When completed 'Uploaded x tracks' will appear 3. Analyzing (T) // Different from step 1. and needed as well 4. Deleting track x of y ... (K) // When completed, 'Deleted x tracks' will appear 5. Downloading track x of y ... (K) // When completed, 'Downloaded x tracks' will appear. |
|
|
3) partialy fixed in build 57 - some issues with track counters and progress bars remain |
|
|
3) fixed in build 59 |
|
|
3) Few tweaks: a) The tasks that are only scheduled and not yet in progress are shown (e.g. 'Downloading', while MMA is still yet 'Uploading'). They'd better be hidden. b) Processed tasks have yellow header. I think that rather the task in progress should have yellow header, the processed one should have gray. c) 'Successfully synced...' message is shown after the sync. It can obscure e.g. # of downloaded tracks, etc. I think that a 'Close' button directly in the activity would make more sense. Not too important though... |
|
|
1) fixed in build 61 2d) fixed in build 61 3b) fixed in build 61 Re 3a) Do you really think it will be better ? It was my intent to display it for the user to have better overview about the progress |
|
|
2b) fixed in build 61 3c) fixed in build 61 i.e. I am waiting for feedback re. 3a 4) I don't understand the wording here: Delete tracks from the device? [ ] Delete only files from the sync list that aren't selected [ ] Delete all media files besides those selected in the sync list I kow that current wording is wrong but I don't think that the suggested wording reflects the functionality. I will explain it. It is also related to current mail conversation about wifi sync: I. Delete unsynchronized It deletes items that are not in sync list but previously was synchronized. Typically it means they are in MMW library. But there are other cases too. User can delete these files from MMW library that was on sync list, so they aren't present on the sync list after that. These files will be deleted too by this process. II. Delete unknown It deletes files that wasn't never synchronized with MMW. So files that are not presented in MMW library. Typically when bi-dir sync is disabled. So these tracks are not uploaded and synced. It deletes only tracks that are stored in folders selected in "Auto-sync (--> Library)" tab on MMW. If no folder is selected, it deletes all unknown files. |
|
|
Re: II. Delete unknown: I think that frolders from Device->Options->File Locations tab should be taken into account (including other media files there) to be consistent with USB sync. Otherwise I agree that the strings are confusing, what about using something like these (I use in MM8): [ ] Delete files no longer on the sync list [ ] Keep only files presented on sync list ? But Marek is true that it is related to current mail conversation about wifi sync and that these strings should wait until there is a conclusion/agreement. Also tooltips/description text of these checkboxes should indicate that it deletes files only from the media locations (Options->File Locations in MMW) |
|
|
3a) I see the biggest problem in the fact that we show a string 'Downloading' while there isn't any download in progress yet. So, should we want to keep it there, we'd at least have to change the wording. However, since I don't think there's any particular need to show these items, hiding them seems to be the easiest solution. 4) I agree that the original Rusty's suggestions are somewhat hard to read/understand. I'd prefer to have some short and clear titles, followed by an explanation in a smaller font, e.g. [ ] Delete unsynchronized - Deletes tracks that were previously synced by MM, but are no longer in the sync list |
|
|
Re. 4) Here's a revised proposal (note--it also includes changes to other strings): Auto-sync to the server [ ] Sync tracks from this device to the server Auto-sync to this device Choose which items to sync from the server to this device Delete items that are unselected [ ] Deletes files and playlists that are explicitly deselected in the Auto-sync list, from the device Delete all other media files [ ] Also deletes all files and playlists that don't exist on the server, from the device |
|
|
3a) fixed 4) wording fixed. Indention not implemented yet. But please confirm following: Should I disable "Delete all other media files" when "Delete items that are unselected" is unselected? |
|
|
4) yes |
|
|
Re. 4) this maps directly to the settings in MMW--it's just presented differently. |
|
|
4) To be fixed. 5) Auto-conversion (tested on video) results in some UI weirdness, most notably: 5a) For some time (like 2 minutes) MMA shows 'Downloading {previous audio track}...' even though it was apparently waiting for auto-conversion result of the video file. 5b) MMA Sync screen and tray info sometimes aren't in sync - e.g. while MMA already shows that it's waiting for auto-conversion result, the tray still shows that it's downloading the previous audio track. 5c) While waiting for auto-conversion result, it would be nice to show basic info, e.g. Video title or Artist-Title pair for Music. 6) Each time user return to sync UI screen using Tray, it's added to the back stack - it shouldn't when the sync UI is already shown. |
|
|
This hasn't been tagged as resolved, but adding a note re. 4) since it appears to be partially resolved in build 75: Delete items that are unselected [ ] AND Delete all other media files [ ] don't have a parent/child relationship. i.e. Delete all other media files can only be selected if Delete items that are unselected is enabled. |
|
|
4) Fixed in build 93 6) Fixed in build 93 |
|
|
5a) Fixed in build 93 5b) Fixed in build 93 5c) it cannot be done in current implementation. It would be nice to show progress of AC but currently MMA has no info about it. I can discuss it with Ludek. |
|
|
5c) Info about currently converted track is problematic. MMW converts many tracks concurrently. Maybe we can show just the max progress of all currently converted tracks to inform the user when download will continue. Maybe we can lower the priority or create a separate issue for this thing. |
|
|
OK--tagging as resolved in 93. Will evaluate the build and open any new bugs as required. |
|
|
Closing New bugs were addressed accordingly over the new versions |
