- Status Unconfirmed Reopened
- Percent Complete
- Task Type Patches
- Category Applications
- Assigned To No-one
- Operating System All players
- Severity Low
- Priority Very Low
- Reported Version Daily build (which?)
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
Attached to Project: Rockbox
Opened by Alexander Levin - 2008-05-24
Last edited by Yoshihisa Uchida - 2010-06-08
Opened by Alexander Levin - 2008-05-24
Last edited by Yoshihisa Uchida - 2010-06-08
FS#9034 - Alphabetically sort the list of plugins shown after 'Open With...'
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.
And now with the patch
A small correction: always show the splash. Otherwise you get an empty list. And that's confusing.
And now without debugging printf's
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?
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.
For the case that we will do promote this to a feature: attached is viewers.config with entries sorted by their simple names.
reopen this task.
reason: text file is read by the sort plugin instead of the text viewer plugin. then, text file is broken.