Zsh Mailing List Archive
Messages sorted by: Reverse Date, Date, Thread, Author

Re: Bug#527301: _xmms2 jump/_values: suboptimal presentation/sorting



also sprach Clint Adams <schizo@xxxxxxxxxx> [2009.05.06.1926 +0200]:
> zstyle :completion::complete:xmms2:\*:values list-grouped false
> zstyle :completion::complete:xmms2:\*:values sort false

Ooooh!

Which documentation did I not read? I tried
http://zsh.dotsrc.org/Doc/Release/zsh_19.html#SEC124 and the source
code, but I couldn't deduce this gem.

> I have no opinion on how the default behavior should be handled.

I'd say for playlists, grouping makes no sense, and since the
playlist is already passed sorted, sorting should be disabled,
especially since it doesn't do numeric sorting anyway.

My vote's for making your suggestion quoted above default.

On a side note: why is the playlist sorted in reverse order?

  18    -- Deep Purple - Smoke on the Water
  17    -- Deep Purple - Speed King
  16    -- Deep Purple - Anya
  15    -- Deep Purple - Child in Time
  14    -- Deep Purple - Anyone's Daugher
  13    -- Deep Purple - Perfect Strangers
  12    -- Deep Purple - A Twist in the Tail
  11    -- Deep Purple - Black Night
  10    -- Deep Purple - Highway Star
  9     -- Deep Purple - Lucille
  8     -- Deep Purple - Speed King
  7     -- Deep Purple - Black Night
  6     -- Deep Purple - Space Truckin'
  5     -- Deep Purple - Lazy
  4     -- Deep Purple - Strange Kind of Woman
  3     -- Deep Purple - The Mule
  2     -- Deep Purple - Smoke on the Water
  1     -- Deep Purple - Child in Time

I'd suggest

--- /tmp/_xmms2 2009-05-06 21:32:06.000000000 +0200
+++ /usr/share/zsh-beta/functions/Completion/Unix/_xmms2        2009-05-06 21:32:23.000000000 +0200
@@ -66,7 +66,7 @@
   fi
      done
      
-     _values -s ' ' 'playlist items' ${(On)playlistitems}
+     _values -s ' ' 'playlist items' ${(on)playlistitems}
 
 }

In addition.

-- 
 .''`.   martin f. krafft <madduck@xxx>      Related projects:
: :'  :  proud Debian developer               http://debiansystem.info
`. `'`   http://people.debian.org/~madduck    http://vcs-pkg.org
  `-  Debian - when you have better things to do than fixing systems

Attachment: digital_signature_gpg.asc
Description: Digital signature (see http://martin-krafft.net/gpg/)



Messages sorted by: Reverse Date, Date, Thread, Author