View Issue Details

IDProjectCategoryView StatusLast Update
0009330MMANavigationpublic2013-10-28 11:19
Reporterjiri Assigned To 
PriorityurgentSeverityfeatureReproducibilityN/A
Status closedResolutionfixed 
Target Version1.0.2 
Summary0009330: Folder browser
DescriptionCurrently user can only browse the collection by attributes, but many user's would definitely prefer browsing by device folder structure. Also, this would allow browsing of some temporarily inserted memory cards. So, we should create an activity that would browse the internal memory, together with external memory (SD cards or connected mass-storage devices).

Suggested features [added 2/25/2013 by Rusty and supercedes all comments until 2/25/2013]
a) at top level show
- If content exists in multiple locations, show both locations i.e. Internal memory, External memory as top-level nodes
- If content exists in only a single location, show the lowest level directory that is superior to the folders containing that content
b) we can use the UPnP folder icon to navigate down a level. As elsewhere in MM, the title bar updates to show the parent directory.
c) use virtual folder '..' to navigate up a level
d) virtual [all] folder could be useful (nice to have)
e) for any folder/virtual folder containing content, indicate # tracks in the right-hand column (e.g. as in Artists view)
d) for any folder containing content, the following context functions should be available (as is the case for e.g. Artists):
Play, Play Next, Send to playlist, Delete, Edit Properties
e) for individual tracks, the context menu should be the same as elsewhere (Play, Play next, Playlist, Trash, Ringer, Share, Properties)
f) should we also have rename, copy/cut & paste (and or copy to / move to) for folders and tracks--most useful for content that isn't in the library?

Update 3/1/2013:
It will work as follows:
- Library content will be quickly looked up in our DB, i.e. all the features will work and it will be fast
- non-Library content will use android metadata reader (and in the future our own code), will be slower in showing all metadata and some features (e.g. adding to playlists) won't work.
Additional Informationhttp://www.mediamonkey.com/forum/posting.php?f=21&t=68864
ZKL-502489
TagsNo tags attached.
Attached Files
Fixed in build128

Relationships

related to 0010170 closedmarek MMA Restrict which folders are used by MMA 
related to 0010500 closedpeke MMA Playlists: We should allow that Playlist can Contain Playlist as object 
related to 0010731 closedmartin MMA Folder Browser: Playback and navigation enhancements 
has duplicate 0010112 closedjiri MMA Folder Browsing 
related to 0010552 closedLudek MMW v4 Nexus 7, JB 4.2.2, MTP sometimes freezes  

Activities

peke

2012-12-03 11:46

developer   ~0033567

Added from 0010112:

Only thing that would be needed for Browsing is to have Long press Context Menu show with OPEN FOLDER(Default action like single click), PLAY FOLDER, ADD FOLDER

peke

2012-12-13 01:10

developer   ~0033850

Last edited: 2012-12-19 02:54

Added relation to 0010170 as it could be useful that within Folder browser in MMA user can disable check box from Jiri proposal in 0010170:0033840

peke

2013-02-08 23:41

developer   ~0034845

Last edited: 2013-02-08 23:41

There is easier way to implement this faster using Playlist.
Playlist named Folders (Or LOCATION) and use it as browse root

Browsing mockup and features is described at 0010500

peke

2013-02-15 01:16

developer   ~0034950

To clarify, Folder browsing would look like Playlist browsing. But instead of playlist names user would navigate thrum track location structure:

eg. /sdcard/external_sd/Music/Ann Lee - 2 Times.mp3 would be

Solution 1 from device Phone root:
sdcard (1 Folder) -> external_sd (1 folder) -> Music (1 Track) -> Ann Lee - 2 Times.mp3

Solution 2 from device Storage root (MMW Profile root):
External Storage -> Music (1 Track) -> Ann Lee - 2 Times.mp3

NOTE: MMA would not actually access and read folders in real time but either use MMA library to create structure in realtime or have pre-created table containing structure created apron device scanning. Whichever is faster

jiri

2013-04-09 07:02

administrator   ~0035582

Per e-mail and IM discussions, assigning to Martin.

rusty

2013-04-17 20:38

administrator   ~0035655

Last edited: 2013-04-17 20:39

There are a couple of basic issues I noticed with the implementation in build 122:
1) The Title on the home screen is 'Browser'. It should be changed to 'Folders'
2) The icon on the home screen is for UPnP. It should be changed to a folder.
3) The icon that appears when the user goes into the browser node is a folder combined with a list. This looks too busy, so I would suggest that it just be a folder.
4) MM consistently crashes when I click the browser node and then click 'Back'. Numerous debug logs have been uploaded between 4:30 and 4:35 today using an S3.
5) Per discussion over e-mail: sort order within a folder should be
Album, Album Artist, Disc #, ...

rusty

2013-04-18 18:54

administrator   ~0035669

Raised to immediate due to the documented crash. Also, attached a debug log of the crash.

martin

2013-04-29 11:09

developer   ~0035850

Remaining points:
d) virtual [all] folder could be useful (nice to have)
d) for any folder containing content, the following context functions should be available (as is the case for e.g. Artists):
Send to playlist, Edit Properties
e) for individual tracks, the context menu should be the same as elsewhere: Ringer, Share, Properties
f) should we also have rename, copy/cut & paste (and or copy to / move to) for folders and tracks--most useful for content that isn't in the library?

jiri

2013-04-29 14:36

administrator   ~0035852

Note that only

d) for any folder containing content, the following context functions should be available (as is the case for e.g. Artists):
Send to playlist, Edit Properties
e) for individual tracks, the context menu should be the same as elsewhere: Ringer, Share, Properties

should be fixed, the rest could be added as new Mantis issues for future versions.

martin

2013-05-14 23:00

developer   ~0036093

Fixed in build 128.

peke

2013-10-28 11:19

developer   ~0038085

Verified 177