Rockbox

Tasklist

FS#9034 - Alphabetically sort the list of plugins shown after 'Open With...'

Attached to Project: Rockbox
Opened by Alexander Levin (fml2) - Saturday, 24 May 2008, 21:19 GMT
Last edited by Yoshihisa Uchida (Uchida) - Tuesday, 08 June 2010, 11:07 GMT
Task Type Patches
Category Applications
Status Unconfirmed   Reopened
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 0%
Votes 0
Private No

Details

I think this is good since now the plugins in the list appear in the order of their occurrence in viewers.config which is rather random.
This task depends upon

Comment by Alexander Levin (fml2) - Saturday, 24 May 2008, 21:19 GMT
And now with the patch
Comment by Alexander Levin (fml2) - Saturday, 24 May 2008, 21:46 GMT
A small correction: always show the splash. Otherwise you get an empty list. And that's confusing.
Comment by Alexander Levin (fml2) - Saturday, 24 May 2008, 21:48 GMT
And now without debugging printf's
Comment by Paul Louden (Llorean) - Sunday, 25 May 2008, 02:17 GMT
Wouldn't it make MUCH more sense to just fix viewers.config, then leave "custom sorting of the Open With screen" as a feature by modifying viewers.config?
Comment by Alexander Levin (fml2) - Sunday, 25 May 2008, 08:44 GMT
That would indeed make sense. But viewers.config (IIUC) has never been considered as a mean to sort (or to set the order of ) entries in the menu. Just the implementation makes it so. If we promote it to a 'desired' and 'official' feature then yes, editing the file will be more appropriate and flexible approach. But now it's just by coincidence.

BTW: editing the file might be tricky for a normal user because entries can occur many times. If I understand the implementation right, the position of a plugin in the menu list is defined by its first occurrence in viewers.config.
Comment by Alexander Levin (fml2) - Sunday, 25 May 2008, 10:01 GMT
For the case that we will do promote this to a feature: attached is viewers.config with entries sorted by their simple names.
Comment by Yoshihisa Uchida (Uchida) - Tuesday, 08 June 2010, 11:08 GMT
reopen this task.
reason: text file is read by the sort plugin instead of the text viewer plugin. then, text file is broken.

Loading...