View Issue Details

IDProjectCategoryView StatusLast Update
0018922MMW 5Main Panel: Toolbars & Menuspublic2022-04-01 16:05
Reporterpeke Assigned To 
PriorityhighSeveritymajorReproducibilityalways
Status closedResolutionreopened 
Target Version5.1Fixed in Version5.0.3 
Summary0018922: Context Menu: Send To have long loading in case recursive show is enabled on Folders
DescriptionSend To Context Menu have long loading in case recursive show is enabled on Folders eg. till all content is loaded.

This especially problematic when user is browsing NAS devices where showing folder content of shared NAS resource can take 10+ minutes.

Disabling recursively show content, make Send to show immediately

Expected behavior is that default content is shown ASAP and and that rest is loaded in background.

Step to reproduce: Enable recursive folder show -> Browse folders -> Network -> NAS -> Right click Music -> Send to -> LOADING .....
Additional Informationhttps://www.mediamonkey.com/forum/viewtopic.php?p=495601#p495601
TagsNo tags attached.
Attached Files
image.png (5,286 bytes)   
image.png (5,286 bytes)   
Fixed in build2611

Relationships

related to 0018909 closedLudek Recursive button on Toolbar is global (?) 

Activities

Ludek

2022-03-15 12:00

developer   ~0067307

Last edited: 2022-03-15 12:01

Can you elaborate what do you mean by this:
>>Expected behavior is that default content is shown ASAP and and that rest is loaded in background.<<

What do you mean by "default content" ? I think that MM5 needs to load whole content to be able to send it.

BTW: For moving whole folder without loading its content you can right-click it in the folder grid > Send to > ...
image-2.png (17,214 bytes)   
image-2.png (17,214 bytes)   

Ludek

2022-03-17 14:02

developer   ~0067327

BTW Could be resolved by Barry's suggestion to not make the "recursive" button state sticky ( 0018909 ) + points at https://www.mediamonkey.com/forum/viewtopic.php?p=495755#p495755
Which is a good point IMO. In MM4 it also was not a global sticky option and user had to always click 'All' subnode to list files from all subfolders.

Ludek

2022-03-17 14:58

developer   ~0067330

Fixed in 2611 together with 0018909
i.e. "Display folder content recursively" button state is no loger sticky/global.

Let's collect feedback on this change...

barrym

2022-03-20 06:48

updater   ~0067350

After reviewing practicality of the new non-sticky behaviour of the Recursive button, I think that the current change is too severe, and will probably be unpopular.
Test:
* List by Album View
* navigate to an artist folder ... turn Recursive on
* navigate to a different Artist folder ... recursive is auto turned off ==> unlikely to be expected nor desired, and since any artist folder will be relatively small, it delivers no benefit either

Could the auto-turn off be enacted in the following circumstances only:
1. Navigation which **changes to** the Folders or Location nodes ... ie. navigation from somewhere else, which effectively is the start of navigation "session" within a specific Location|Folder node
2. Navigation, during a Location|Folder "session" which changes to a different disk root ... ie. to a different disk drive, or to a different network share

I think that this will be less disruptive, and easier to understand, and sill deliver the benefits that were the aim of the change.

peke

2022-03-21 23:00

developer   ~0067353

Assigned for review

Ludek

2022-04-01 16:04

developer   ~0067411

Fixed/tracked in 0018909