View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0015584 | MMW 5 | Main Panel | public | 2019-04-10 17:34 | 2020-03-20 12:07 |
Reporter | rusty | Assigned To | |||
Priority | urgent | Severity | major | Reproducibility | always |
Status | closed | Resolution | reopened | ||
Product Version | 5.0 | ||||
Target Version | 5.0 | Fixed in Version | 5.0 | ||
Summary | 0015584: Views: Subviews should persist on a per-view basis | ||||
Description | Currently, subviews are presented as _view-specific_ options. e.g. 'Related Artists' is an option for 'Album Grid' but not 'Tracklist'. Since the subviews are inherently view-specific, their settings should also be saved per view. e.g. if I disable the 'Info panel' for 'Tracklist' it should remain enabled for 'Album Grid' (i.e. users will Typically want one view with rich metadata/browsing abiliities and another one to manage their collection). | ||||
Tags | No tags attached. | ||||
Attached Files | |||||
Fixed in build | 2195 | ||||
related to | 0015926 | closed | rusty | Improvements to navigation of Playlist/Folder hierarchy |
related to | 0015712 | closed | rusty | "Art View", "Browser View", "Grid View" - why different names for the same view |
related to | 0014086 | closed | michal | Add 'Column browser'/Filter to more views |
related to | 0016052 | closed | michal | Sorting in Playing view causes crash - regression |
related to | 0016391 | resolved | rusty | List views are not configured/saved per-node |
child of | 0015756 | resolved | rusty | 'Manage views' functionality seems broken / incomplete |
|
While I understand that in some cases this approach would be useful, I think that in others it wouldn't. E.g. Column Browser probably works better independent of any view. We could try to implement some customization options, possibly in the Manage Views dialog, but I think it probably isn't worth any significant effort and so we should either close this, or leave it open as a lower priority item and re-evaluate later. |
|
Moving to 5.0.1 for re-evaluation. That said, if we feel that a feature should be active across views, then it isn't really a _sub_view. |
|
Reset to 5.0 for additional discussion. Re. the example of a Column Browser: I disagree--if the user disabled it for 'Simple tracklist' but left it enabled for 'Tracklist' it's because that's what they intended. Here's a case where the current implementation makes no sense at all: Music > Artists > Artist Name [Album Grid] - Subview:Infopanel should be active. But if the user switches to the tracklist then it's presumably because they want a view that doesn't include the Infopane (I certainly don't)! This ends up filling what should be simpler views with content that users don't want. Subviews really should only persist with each view. Also, this is what users will expect based on the current wording. |
|
The following issues originally raised at 0015712 can only be fixed with the resolution of this bug: Music > Albums > AlbumName (List): shouldn't have the info-panel by default (but disabling it shouldn't disable it from the Browser view). Music > Artists > ArtistName (list): Should not have the info-panel enabled (but disabling it shouldn't disable it from the Browser view). Music > Artists > ArtistName (list (by album)): Should not have the info-panel enabled (but disabling it shouldn't disable it from the Browser view). |
|
Per IM discussion, Ludek will implement as described. Then, Petr probably could also add sub-views to be customizable in the custom views. |
|
Implemented as described in 2194. Petr will add sub-views to be customizable in the custom views (it is already tracked as item 2) in 0015756 ). |
|
Reopen for feedback, during testing while trying to enable/disable Info-Panel in various views, MM5 crashed. Pics attached. |
|
Re crash 77950493: It is related to sorting (when sorting threads took the lock for long time (> 10s) and thus it asserted) Petr is already making imrovements, details discussed over IM |
|
Fixed |
|
Unable to replicate in 2195 Cloase after second Verification. |
|
Verified 2195. |