View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0002701 | MMW v4 | Main Panel/Toolbars/Menus | public | 2006-11-21 19:29 | 2011-11-12 10:19 |
Reporter | jiri | Assigned To | |||
Priority | urgent | Severity | minor | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | 2.5.5 | ||||
Target Version | 4.0 | Fixed in Version | 4.0 | ||
Summary | 0002701: Context toolbar + rationalize commands between menu and toolbar | ||||
Description | By working with iTunes I got an idea of context toolbar. In iTunes, depending on where you are, you have some buttons directly accessible (for example, if you select a CD drive, 'Import CD' button appears). I wonder whether this wouldn't be useful in MM too. There would be a toolbar with commands that make sense in the current context. E.g. If user goes to Podcasts node, 'Update Podcasts' action would appear. A possible way of implementing a context-sensitive toolbar would be to set the context based on which node is selected. This is a bit different than the new office/WMP UI which set the context via a large button in the toolbar, however, this could work very well IFF the shorcuts to the root nodes were always available (bug 0001346) so that users would always have a means of switching contexts. Depending on what node is selected, the toolbar commands would change accordingly: a) when library nodes are selected, have it function as it does today b) when other nodes are selected, have it function as follows: Note: this is more of a braindump than a spec. It needs to be refined. If we agree on the idea, then I'll spend more time on it. All contexts: Primary: Search Secondary: Sleep, Add/scan, Maintain Library, Clear Library, Library nodes OR Tracks displayed: Primary: Burn, Synch Selected, Play Now, Play Next, Play Last, Playlist, Properties Secondary: Send to, Find more from same, Get Info/Buy, Cut, Copy, Paste, Remove, My Rating, Auto-Tag Amazon, Auto-Tag Filename, Auto-Organize, Advanced Tag Management Functions, Save to Virtual CD, Save Preview, Analyze Volume, Level Volume, Categorization, Locate moved/missing, Create reports, Export to .m3u View: Browse, Show Art/Details/Both Library/Files to Edit/Multiple-Artist Albums: Same as Library + Primary: Change Artist Album Library/Files to Edit/Dead Links: Same as Library + Primary: Locate Moved/Missing tracks Unleveled: Same as Library + Primary: Analyze Volume Unsynchronized Tags: Same as Library + Primary: Synchronize Tags Duplicate Titles: Same as Library + Primary: Remove duplicates Playlist Nodes: TBD No Contexts: Add/rescan Locate Moved/missing tracks Podcast node: -Subscribe to new podcasts (directory) -Update podcast(s) -Podcast Settings NP: Node -Visualization CD Node: -Rip (this only makes sense if CD nodes appear at the root like iPod devices do) Radio Node: -Save station Synch Node: -Synch Selected -Auto-synch -Remove Device -Configure | ||||
Tags | No tags attached. | ||||
Fixed in build | 1370 | ||||
related to | 0001346 | feedback | jiri | Tree is crowded and nodes are sometimes inaccessible |
parent of | 0007769 | feedback | Ludek | Irrelevant / too many podcast update buttons |
has duplicate | 0003806 | closed | jiri | 'Subscribe to new podcast' available in the main menu |
related to | 0002624 | feedback | jiri | Rip CD: Users are often unclear how to edit the metadata |
related to | 0002825 | closed | petr | Vista-style hierarchy browser |
related to | 0007710 | closed | petr | Send to / Tools buttons don't indicate that they contain submenus |
related to | 0008325 | closed | Ludek | Rip CD/FreeDB Lookup only available in CD node |
Not all the children of this issue are yet resolved or closed. |
|
Assigning to Rusty for a review. |
|
posted some thoughts on this. Let me know what you think. |
|
This structure is certainly an option too, but my original intention of this feature was: 1. To keep the current toolbars pretty much as they currently are, i.e. with a fixed structure (in future possibly configurable), so that user can use them easily regardless of the active node. 2. To add a new toolbar (actually I'm not sure if an ordinary toolbar, or something positioned differently, like in status bar??) that will _only_ have really context specific commands and would even disappear for some nodes. Examples of use are very similar to what you described above, e.g.: Podcast node: -Subscribe to new podcasts (directory) -Update podcast(s) -Podcast Settings CD Node: -Rip Radio Node: -Save station Synch Node: -Synch Selected -Auto-synch -Remove Device -Configure |
|
After reviewing podcasting functionality, it is obvious that something along these lines is needed--at least for that functionality, since currently, podcast commands are only available via tree context menus. A suggested approach would be to have a 'Podcasts' toolbar appear whenever the node is active. It would contain the following: -Subscribe (to New Podcast) -Update Podcasts -Add Directory -Update Directories |
|
I think the main point, in my opinion, of context sensitive toolbars, is to keep the toolbars clutter free - only showing what is relevant to where you are... this of course, is obvious. However, in the goal to make it clutter free, even the context menus mentioned can use some cleaning (showing only what is MOST used). For instance.. - "cut", "copy", "delete", and "paste" have 'well known' keyboard shortcuts, are in the drop down menus and their actions are usually achieved by dragging and dropping anyway. So these should be culled from the toolbar (at least by default). - The "play selected track", "play selected track next", and "play selected track last" buttons can all be ditched. (MM has a habit of catering for advanced users in some ways, but then providing overly simplistic methods too - ie. everyone knows how to add tracks to the playlist... these 3 options are also available with a right click.. they're not needed as buttons to clutter the layout). - "create playlist", and "create autoplaylist" should be added in the playlist context toolbar so you can create a blank playlist without having to first choose a song to send to it. - Categorize options should only show if an actual track(s) is selected (and in the currently focussed pane) I 'think' the "start/stop visualization" button can be ditched as many player skins can provide it (in a position where it will be more relevant) - and if a skin doesn't put it in.. It is in a drop down menu and will take a user 2 seconds to learn the shortcut if they plan to use it alot. - "save selected track to virtual cd", "copy selected track to virtual cd", "move to folder", and "copy to folder" are all intermediate functions and are readily available from right clicking on a track. They aren't options that are constantly used or expected.. they should all "not" make an appearance in the context toolbars (by default at least). |
|
per jiri's feedback, as part of this issue, we should review which commands are not available via Main Menu and context menu and make them consistently available. e.g. View Album Art Album Browser Sync Podcast etc. |
|
We can split this bug up into 2 parts: a) Ensure user is able to access commands (e.g. right now some users can't find how to access Podcasting functionality) b) Optimize the UI so that irrelevant commands don't clutter the UI in certain contexts. If we try to resolve a) right now, we could make the following changes (all are designed using existing strings): Main Menu Changes ================= 1)Add 'Sync Device' (under Burn CD) with submenus identical to those in the toolbar >Sync Selected >Auto-Sync -------------- >x Device A > Device B -------------- >Options An alternative would be: Sync Device... --> Dialog appears: ------------------------------------- Sync Device ------------------------------------- (o) Device A ( ) Device B (o) Auto-sync ( ) Sync Selected [Options] [[OK]] [Cancel] ------------------------------------- I prefer the second option, as it could also be used for a Sync Device button in the tree (and even to simplify the current sync device button in the toolbar) 2) In the View menu, under Party Mode, add: ... Party Mode ------------ Show Details Show Album Art Show Album Art + Details ------------ Configure Visualization .... 3) In the Edit Menu add the following below Add AutoPlaylist: Podcasts >Subscribe to new Podcast >Update all Podcasts >Podcast directories [[note: this is just a shortcut to the first directory]] >Options Toolbar Changes: ================= 4) Add a new Podcasts button whose behavior is configurable Podcast ^ The following are the configurable options + Global options o Subscribe to new Podcast Update all Podcasts Podcast directories -------------------- Options Updates to the Shortcuts Toolbar ================================= 5) The following structure is designed to ensure functionality is available even when portions of the toolbar isn't visible: Back, Forward, Collapse, Now Playing, Artist, Genre, Playlist, Podcast, My Computer, Net Radio Note: I wouldn't mind moving the Back/Forward buttons elsewhere, but I'm not sure of a better location. Updates to the Context Menu =========================== 6) Add Analyze Volume and Level Track Volume under to Auto-Organize Files |
|
Assigning to Ludek, re. 1) I'm for the first option for now, we can implement the second for some future version. |
|
Items 2 and 3 added in build 1092. |
|
Items 1,4,6 implemented in build 1093. |
|
And finally the item 5 implemented in build 1094. So, all the items are implemented in build 1094. Re: 4) I implemented it slightly different way. Try to review, if you don't like it, re-assign to me. Re: 5) I reorder it, but there are missing the hovering hints upon the new buttons, because I am not sure whether we have strings like "Go to the Podcasts node". I guess that we haven't. |
|
Verified 1094. Assigning to Rusty to spec changes for 3.01. |
|
another issue to consider: MM looks a little strange when there are greyed out icons then active icons then greyed out icons. i.e. disabled functions should be clustered. |
|
Marking as fixed, since any remaining issues have been resolved by the new UI in MM4. |
|
Checking Podcasts I noticed: 1) Update Podcast and/or Update Podcasts is missing from toolbar 2) Edit is enabled when selecting Podcast node, but doesn't do anything 3) Burn (when just selecting Podcast node) will select all files even non-downloaded ones, didn't test what happens when burning 4) Once you select an episode the Download button remains enabled when switching to a different Podcast. If you select a track in the NP and then switch Podcast nodes Download remains disabled. Using this Download button on an already downloaded episode will throw an Unknown Protocol error. 5) Edit can be enabled and not work when selecting Now Playing or Collection. 6) Tools and Send To buttons fail to indicate they have a submenu. 7) Clicking on an empty area in the Filelisting will have Burn and Edit enabled. Edit fails silently, Burn fails to have any tracks preselected. 8) In 7's scenario Tools will have Advanced Tag Management enabled while child nodes are disabled (http://www.mediamonkey.com/forum/viewtopic.php?f=6&t=56162). |
|
Assigning to Ludek to fix the items that are clear, please assign anything unclear to Rusty for a review. |
|
Some clarification re. some of the UI issues: 1) Update Podcast should be added to the _Tree_ (similar to the Sync button that appears next to a device in the tree). 2) To clarify: The Edit button should be shown for this node, but it should be disabled if no track is selected. 3) I would suggest that it would be correct behavior if MM would either: a) silently fail for non-downloaded tracks b) download all selected tracks prior to burning 6) Moving this to a separate issue. 4/5/7/8) Leaving for Ludek... |
|
All is fixed in build 1370. Re: 3) Fixed using solution a) Re: 5) I would rather hide the 'Edit' in the context toolbar (if it is disabled)? Re: 7) 'Edit' is fixed, 'Burn' should be always ebnabled because user can select desired tracks directly in the Burn Wizard. |
|
There was an AV on MM startup (100% reproducible here). I fixed it, but leaving open for Ludek in order to review the fix and the overall workflow there. |
|
Peke, why is this reopened? |
|
I assumed it needs more tests Verified 1458 |