Bug#560106: Spellchecker: Language entries are unsorted, some duplicated and there are too many entries.

[Andrés Hamann]

Package: iceweasel
Version: 3.5.5-1
Severity: normal


The languages list for the spelling is extremely long and has no order.
I find myself using different languages and changing the writing
language often, therefore I have the following dictionaries installed:
myspell-ca, myspell-de-de, myspell-en-gb, myspell-en-us, myspell-es,
myspell-fr.
After last update, the "Languages" menu in iceweasel lists around 17
"Spanish / " dictionaries (main annoyance), this might be a bug in
myspell (please confirm if you agree) however iceweasel should also be
able to handle this better.
Also some of the entries are duplicated, having the list alphabetically
ordered and without duplications would be a good start.

This bug is related to this bug:
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=557539
Which is marked as fixed in upstream:
https://bugzilla.mozilla.org/show_bug.cgi?id=528893

But that fix has no relation to the problem of duplication and lack of
order.

I wrote a quick patch that seems to work, to order the entries and
ignore duplicated ones (that increases menu's usability), however I'm
not sure that it's the optimum nor the preferred way to do it but
hopefully it will give someone the idea to make it right.
A patch making submenus for each language with more than 3 entries or so
would be the ultimate solution, but I don't really know how to do that.

You can find attached the mentioned patch, written in Debian Squeeze,
with 2.6.30-2-amd64 kernel over the file
"toolkit/content/inlineSpellCheckUI.js" from the sources downloaded with
"apt-get source iceweasel" and this sources.list line:
deb-src http://ftp.es.debian.org/debian/ squeeze main contrib non-free


-- System Information:
Debian Release: squeeze/sid
  APT prefers testing
  APT policy: (900, 'testing'), (600, 'stable'), (500, 'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.30-2-amd64 (SMP w/2 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages iceweasel depends on:
ii  debianutils                   3.2.2      Miscellaneous utilities specific t
ii  fontconfig                    2.6.0-4    generic font configuration library
ii  libc6                         2.10.2-2   GNU C Library: Shared libraries
ii  libglib2.0-0                  2.22.3-1   The GLib library of C routines
ii  libgtk2.0-0                   2.18.3-1   The GTK+ graphical user interface 
ii  libnspr4-0d                   4.8.2-1    NetScape Portable Runtime Library
ii  libstdc++6                    4.4.2-3    The GNU Standard C++ Library v3
ii  procps                        1:3.2.8-2  /proc file system utilities
ii  psmisc                        22.8-1     utilities that use the proc file s
ii  xulrunner-1.9.1               1.9.1.5-2  XUL + XPCOM application runner

iceweasel recommends no packages.

Versions of packages iceweasel suggests:
pn  latex-xft-fonts      <none>              (no description available)
ii  libkrb53             1.6.dfsg.4~beta1-13 Transitional library package/krb4 
pn  mozplugger           <none>              (no description available)
pn  ttf-mathematica4.1   <none>              (no description available)
ii  xfonts-mathml        3                   Type1 Symbol font for MathML
pn  xprint               <none>              (no description available)
ii  xulrunner-1.9.1-gnom 1.9.1.5-2           Support for GNOME in xulrunner app

-- no debconf information