View Issue Details

IDProjectCategoryView StatusLast Update
0015464MMW 5Tagging / organizing (properties / auto-tools)public2019-09-26 19:20
Reporterrusty Assigned To 
PriorityurgentSeverityfeatureReproducibilityalways
Status closedResolutionreopened 
Product Version5.0 
Target Version5.0Fixed in Version5.0 
Summary0015464: Auto-tag: dialog can't be used effectively while lookups are in progress
DescriptionCurrently, when lookups are in progress, the auto-tag dialog can't be used because tracks are continually moving around (due to sorting/grouping) as tracks are looked up. This makes it difficult to:
- know which tracks have been updated and which haven't
- review tracks in their entirety because tracks get moved prior to the track that's being reviewed
- edit/verify properties of tracks that have been looked up because they get moved around even as they're being edited

Possible approaches discussed to solve this:
a) don't auto-group tracks. Rejected because the result would be unfriendly for grouped tracks (albums).
b) don't display the auto-tag editor until the metadata lookup is complete. i.e. just show a status bar indicating x/y lookups complete as the lookups progress) and then display the UI once the batch is looked up, but with placeholder artwork (artwork lookups should only begin _after_ metadata lookups are completed because they can be looked up even as the user starts performing edits since it doesn't affect grouping, and delaying the artwork until metadata is complete will speed the metadata lookups). This could have the additional benefit of keeping the user's focus on the still-useful MM UI (instead of the auto-tag dialog which isn't very useful as the lookups proceed).
c) pause auto regroup when any combo is expanded ... so it will not regroup when you start editing (petr's suggestion--not sure I understand)

This isn't critical--but if taking the approach of b) would also solve the problem and improve performance, it would yield a better experience. We should only include in 5.0 if this is easy/low risk.
TagsNo tags attached.
Fixed in build2202

Relationships

related to 0015344 closedpetr Auto-tag: Dynamic Sorting makes it difficult to edit tracks 

Activities

petr

2019-02-15 10:16

developer   ~0052639

Approach c is already implemented (there were regression in 2159). Simply when user click on any field to show editor (like title field or album field etc.) auto-tag lookups will continue BUT tracklist stops regrouping until editor will be closed. When editor will be closed without any change, then regrouping continue ... when user makes any change, then toast with info about manual regroup appear and auto-tag stop regrouping till user presses F5.

rusty

2019-02-15 21:19

administrator   ~0052648

Last edited: 2019-02-15 21:49

Tested 2160 and it's better, though there are still issues. If the user edits tracks as lookups are happening:
1) then when they get to the 'bottom' there is a single album art image with e.g. 30 tracks, and the user may not realize why (it looks like a bug, even though F5 resolves it).
2) When the user then presses F5, the tracks are resorted/regrouped and so the user has to restart reviewing/editing tracks from the first one, and this ends up not saving much time.

If we continue with this approach, then we can solve both issues by:
a) showing the F5 popup again when the lookup is complete (assuming the user has editing tracks without pressing F5)
b) when grouping/sorting is 'paused' don't group all tracks with a single album. Instead show the album art that's looked up for each track. That way the user can continue reviewing all of the tracks without having to press F5.

petr

2019-03-18 21:24

developer   ~0053012

Last edited: 2019-03-18 21:27

Item a implemented.
re b) but by default tracks are grouped by albums (including tracks with unknown album) and after edit we shouldn't regroup to all tracks without album are separated.

rusty

2019-07-24 19:21

administrator   ~0054200

In build 2186 it's still difficult to edit tracks as they're being looked up because as soon as a track is edited, newly looked up tracks don't appear sequentially in the list. See below to better understand:
https://www.screencast.com/t/PfdDUOx8Czz

petr

2019-09-24 17:35

developer   ~0054835

Last edited: 2019-09-24 17:35

auto grouping go suspend once you change any value ... you think auto grouping should be enabled once editing is done, but without resorting (so tracklist will not move) ?

rusty

2019-09-24 21:16

administrator   ~0054843

I'm saying that once the user starts making edits:
- All tracks that are looked up should appear in the same order in which they appear
- No additional tracks should be auto-grouped
- Tracks that are looked up or manually edited to a new/different album should be removed from their existing group so that Album Art can be displayed for the track in question. In such cases the track should appear immediately following the current group within which it was contained.
- No additional tracks should be auto-sorted (including tracks that are looked up or manually edited)
- F5 (reset) would auto-group / auto-sort the list and re-enable auto-grouping/sorting until the user makes another edit

afaik, this is the only way to make editing of the dynamically updating tracklist usable.

petr

2019-09-26 18:13

developer   ~0054865

Fixed

rusty

2019-09-26 19:20

administrator   ~0054866

Verified 2202.