View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0015878 | MMW 5 | Sync | public | 2019-08-13 11:58 | 2019-11-24 01:02 |
| Reporter | Ludek | Assigned To | |||
| Priority | urgent | Severity | minor | Reproducibility | have not tried |
| Status | closed | Resolution | reopened | ||
| Product Version | 5.0 | ||||
| Target Version | 5.0 | Fixed in Version | 5.0 | ||
| Summary | 0015878: Device config clearness issues | ||||
| Description | There are various device config clearness and tooltips issues raised here: https://www.mediamonkey.com/forum/viewtopic.php?f=30&t=94739&p=460800#p460394 and here: https://www.mediamonkey.com/forum/viewtopic.php?f=30&t=94739&p=460800#p460798 to review and update. Some of them are fixed in 2192 -- mainly added the missing tooltips from MM4 etc. | ||||
| Tags | No tags attached. | ||||
| Fixed in build | 2215 | ||||
|
|
# 1: Re. apparent conflict re. Auto-sync of Playlists and Options > Playlists > Sync Playlists: this option and many others in Options: File Locations/Tagging/Playlists are relatively advanced (used only for setting profiles) and not required for 90% of cases (which involve Android or Apple devices). Considering this, we might want to consider moving all Profile-specific Settings directly into the Profile settings screen so that it's clear which items are part of a Sync Profile, and so that these settings wouldn't even be looked at in the case of pre-configured profiles. What do you think? # 2: I agree that it would be useful to have a summary that showed all selected items. Possible future enhancement. # 4 Tooltips: A) Deletes files from the device if they're not on the Auto-Sync list. In general, this should be enabled, but be aware that it can delete files from the device if you've added them to the device via another computer or if you've modified file tags on the device but not in the Library. --> Deletes remote content that isn't selected in the Auto-Sync list (either all unselected content or only unselected content that exists in the local database). Deletion occurs after remote content has been synced (if that option is available/enabled). In general, deletion should be enabled to prevent duplicates on the remote location, but the option to delete 'all' unselected content should be used carefully if the remote location contains new/modified content that isn't in the local database. B) MM4 tooltips are still missing (in 0002192) for the Auto-Sync (-->Library) tab C) Device profiles contain default settings that allow adevice or storage location to sync correctly. They don't include user-settings such as which content will sync --> Sync Profiles contain default settings for a device or storage location to sync correctly. They don't include user-settings such as which content will sync. Also, I'd suggest that this text should appear at the top of the screen (instead of at the bottom), replacing "Profiles for various devices and storage locations can be imported via the MediaMonkey addons database:" At the same time, for consistency: Settings profile --> Profile settings C)ii Reset Settings...: Resets this sync profile to its default settings C)iii Are you sure you want to reset this profile settings? --> Are you sure you want to reset this profile to its default settings? D) This causes the playlist to use a relative path to the files --> Relative paths store file links in a playlist in relation to playlist's location, allowing the playlist and content to be moved to other locations without breaking the links. E) Level volume when synchronizing audio: Adjusts the volume of files to the chosen level (lossless only for MP3 files). Mainly for older devices since since most devices can adjust volume on the fly based on Replay Gain tags. Minor UI changes: F) Confirm deletion (recommended) --> Show confirmation dialog (Tooltip: Shows a confirmation dialog with a list of the files that will be affected) G) Confirm updates (recommended) --> Show confirmation dialog (Tooltip: Shows a confirmation dialog with a list of the files that will be affected) # 8: If the phone goes offline, and the associated nodes disappear, 'tracklist' focus should probably switch to whichever node comes in focus (I suppose it would make sense for focus to switch to the Parent of the inaccessible node. Note: the logic for this should probably be the same logic that is used when other nodes disappear. e.g. if the user deletes a node that is currently in focus. # 9: filter to show only MM-synced content [Possible future enhancement] |
|
|
Mostly fixed in 2195. Just re #1 and your question re moving Profile-specific Settings directly into the Profile settings screen. I guess that all the profile specific settings already is under the [Options] tab, so I just renamed the tab to [Profile settings] Let's review this in 2195 -- the only strange thing now is that the [Profiles settings] tab has a 'Profile settings' sub-panel that includes the Import/Export buttons. So we might want to rename the panel to 'Import/export' or something like this ? |
|
|
All settings on the [Options] are part of the profile (including auto-conversion). BUT while checking this I found that more than this is part of the profile. Also some user specific settings like whether to scan on connect, folder to sync content back, whether to delete content etc. are part of the profile! So I guess that we should just adjust this and ensure that we save/load (from *.MMDC) just all the items that are on the [Options] tab -- and not any others (as all the other settings on the other tabs are user specific settings). Do you agree? |
|
|
Yes--it should be obvious which settings are part of the 'profile'. The only question is whether some of the settings you describe _are_ profile-specific and not in the 'profile settings'. Specifically, - are there 'devices' for which additional folders need to be pre-configured in the 'Auto-Sync (--> Library)' settings ? - is 'Auto-sync file metadata from the device to the PC' a 'Device profile'-dependent setting? BTW, I also agree with your previous comment that we shouldn't have 'Profile settings' both in the tab and in the panel. Perhaps: Tab: Device/Service profile Panel: Manage settings |
|
|
Fixed in 2196. i.e. corrected the wording + adjusted loading/saving *.mmdc to keep just the values under the [Device/Service profile] tab. I believe that 'Auto-Sync (--> Library)' settings is user-specific and shouldn't be part of the profile, but we can adjust later (if we find it as not desirable) |
|
|
Verified 2197 Verified Wording, much clear now, also verified Import/export that as Ludek said no user specific settings are Exported/imported |
|
|
Minor tweak. Currently the tabs indicate: Auto-Sync (-->Device) and Device/Service profile Would it be possible to use text that matched the device/service? e.g. For devices: Auto-Sync (-->Device) and Device profile For services: Auto-Sync (-->Service) and Service profile |
|
|
Shouldn't we use the word "Target" (as we use in the other places)? So it would be "Auto-Sync (-->Target)" and "Target profile" ? EDIT1: Note that for cloud profiles we don't have the [Auto-Sync (-->Service)] tab at all, we have only [Remote content], [Library Content], [Device/Service profile] as suggested elsewhere. EDIT2: I see that we have already eliminated also the "Target" terminology and replaced it by the device/service name, see 0014983 So with that in mind the tabs should look like [Remote content], [Library Content], ['Google Drive' profile] -- for clouds ('Google Drive' in this case) [Summary],[Auto-sync (-->Device)],[Auto-sync (--> Library)]['G6' profile] -- for devices (LG G6 phone in this case) ?? But maybe for the tab name we shouldn't use the exact device/service name, but rather generic 'Device' and 'Service' as you suggested. i.e. [Service profile] for cloud services and [Device profile] for device storages. => Adjusted in 2198 ----------------------------------- Another choice is to use just generic [Sync profile] as the tab name? This would make most sense as the heading text below indicates: "Sync profiles contain default settings that allow a device or storage location to sync correctly. They don't include user-settings such as which content will sync." Thoughts? |
|
|
"Another choice is to use just generic [Sync profile] as the tab name?" Yes--that's probably simplest. |
|
|
OK, changed to [Sync profile] in 2198 |
|
|
Verified 2200. |
|
|
Re-opened: per discussion here https://www.mediamonkey.com/forum/viewtopic.php?f=30&t=95379#p462942 (see item #8 there) The "Manage settings" does not seem to be the best wording. Should be rather something like "Import|Export profile" or just "Profile" ? and rather be moved to the bottom of the list, because in most cases it is not required. |
|
|
I like the idea of moving it to the bottom since it's rarely used. As far as naming it: 'Manage settings' isn't great, a bit too generic, but passable 'Profile' is even more generic. 'Import / Export profile' works, but is kind of wordy and will necessarily expand the width of the tree 'Import / Export' is misleading because it could be interpreted as referring to tracks 'Save / Load' could also work but might also be misleading (same as 'Import / Export' I've seen 'Maintainance' used for this type of functionality as well, but it seems too technical to me. Overall, I'd lean towards keeping 'Manage settings', but I don't feel that strongly about this one way or the other. |
|
|
OK, I would also say that having 'Manage settings' as the last panel seems fine. Fixed in 2212 |
|
|
Verified 2212. |
|
|
Re-opened: We should improve clearness for the very old feature of generating device playlists for albums/artists/location, details discussed here https://www.mediamonkey.com/forum/viewtopic.php?f=30&t=95428&p=463128#p463127 (see item #1) Maybe we could change the wording like this: Generate playlists on the remote target for: ...[ ] Selected artist items ...[ ] Selected album items ...[ ] Selected folder items => changed in 2214 |
|
|
Corrections for grammar / consistency: Generate Playlists on the remote target also for: --> Also generate Playlists on [Device Name] for: Selected artist items --> Each selected artist Selected album items --> Each selected album Selected folder items --> Each selected folder Tooltip: Create Playlists to more easily navigate tracks on a device. --> Create Playlists to more easily navigate tracks on a device. This is useful for older players or car stereos that don't have a media database. |
|
|
I am afraid that "Each selected artist" might indicate that the artist may be selected also as a part of a playlist, genre or a collection item. So by using "Selected artist items" I wanted to say that artist _item_ needs to be selected on the sync-list (not a genre item or playlist item). Another (and maybe even clearer) alternative is "Selected artist nodes" ? |
|
|
Sounds good. |
|
|
Fixed in 2215 |
|
|
Verified 2215. |
