View Issue Details

IDProjectCategoryView StatusLast Update
0015584MMW 5Main Panelpublic2020-03-20 12:07
Reporterrusty Assigned To 
PriorityurgentSeveritymajorReproducibilityalways
Status closedResolutionreopened 
Product Version5.0 
Target Version5.0Fixed in Version5.0 
Summary0015584: Views: Subviews should persist on a per-view basis
DescriptionCurrently, 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).
TagsNo tags attached.
Attached Files
bug15584_1.jpg (142,454 bytes)   
bug15584_1.jpg (142,454 bytes)   
bug15584_2.jpg (108,601 bytes)   
bug15584_2.jpg (108,601 bytes)   
Fixed in build2195

Relationships

related to 0015926 closedrusty Improvements to navigation of Playlist/Folder hierarchy 
related to 0015712 closedrusty "Art View", "Browser View", "Grid View" - why different names for the same view 
related to 0014086 closedmichal Add 'Column browser'/Filter to more views 
related to 0016052 closedmichal Sorting in Playing view causes crash - regression 
related to 0016391 resolvedrusty List views are not configured/saved per-node 
child of 0015756 resolvedrusty 'Manage views' functionality seems broken / incomplete 

Activities

jiri

2019-04-11 14:04

administrator   ~0053182

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.

rusty

2019-04-12 15:13

administrator   ~0053200

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.

rusty

2019-07-24 21:00

administrator   ~0054202

Last edited: 2019-08-09 14:54

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.

rusty

2019-08-09 16:45

administrator   ~0054305

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).

jiri

2019-08-28 13:21

administrator   ~0054433

Per IM discussion, Ludek will implement as described.

Then, Petr probably could also add sub-views to be customizable in the custom views.

Ludek

2019-08-29 12:58

developer   ~0054453

Last edited: 2019-08-29 14:39

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 ).

peke

2019-08-31 21:51

developer   ~0054494

Reopen for feedback, during testing while trying to enable/disable Info-Panel in various views, MM5 crashed.

Pics attached.

Ludek

2019-09-02 07:41

developer   ~0054500

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

petr

2019-09-04 17:12

developer   ~0054512

Fixed

peke

2019-09-07 23:19

developer   ~0054540

Unable to replicate in 2195

Cloase after second Verification.

rusty

2019-09-08 03:18

administrator   ~0054545

Verified 2195.