Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category User Interface
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by roolku - 2006-04-02
Last edited by barrywardell - 2007-07-28

FS#4994 - BUGFIX: remove inappropriate "open with"

The patch removes an inappropriate “open with” option from the context menu when in WPS or DB view. The option only makes sense for a file in the file tree.

Closed by  barrywardell
2007-07-28 12:15
Reason for closing:  Out of Date
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

This doesn't seem to be a problem any more with current SVN.

Why is that option inappropriate? It is used to (for example) open a currently playing/selected mp3 file in the splitedit or vbrfix plugin, or the currently playing/selected WAV file with either the MP3 or WavPack encoder.

Well, the reason I spotted the issue was that if you long-click on genre/artist etc in the tag DB view, you get the lonely option 'open with' which leads to an error regardless of what you select and definitly should not be there. I still consider open_with an action to be performed on a file while browsing and not "while playing". With the same reasoning you could have delete and rename in the WPS context menu (which I would find much more useful). With the addition of eq & pitch the context menu becomes rather cluttered.

BTW: the behaviour was introduced with http://www.rockbox.org/viewcvs.cgi/apps/onplay.c?rev=1.73&view=markup

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing