428401 - Port |Bug 367533 – Dictionary Options tab has issues when there is no dictionary| to SeaMonkey

Status: NEW

(SeaMonkey :: Preferences, enhancement)

Description

[Serge Gautherie (:sgautherie)]

Per bug 360488 activity:
{{
Who  	When  	What  	Removed  	Added

[e-mail]  	2007-06-26 17:04:04 PDT  	Depends on  	  	367533
}}

***

While the TB patch is rather trivial,
it seems that in order to be applicable to SM,
we would have first to move the "Get more dictionaries" feature,
from inside the menulist to outside of it, as it is in TB.

See
<http://bonsai.mozilla.org/cvsblame.cgi?file=/mozilla/mailnews/compose/prefs/resources/content/pref-composing_messages.js&rev=1.6&mark=126,133,138-139#113>
<http://bonsai.mozilla.org/cvsblame.cgi?file=/mozilla/mailnews/compose/prefs/resources/content/pref-composing_messages.xul&rev=1.52&mark=79-80#61>

Did I understand it right ?
Is that what we want to do ?

Comment 1

[Ian Neal]

Do you have screen shots showing the problems?

Comment 2

[Serge Gautherie (:sgautherie)]

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9pre) Gecko/2008041001 SeaMonkey/2.0a1pre] (nightly) (W2Ksp4)

I tried to reproduce, per bug 367533 comment 8 steps, with current SM code, and couldn't.

Then, I began to port the code, until I started to wonder what to do about |gLastSelectedLang| and |SelectLanguage()|, which TB don't have.
With the code in that state, I experienced issues like:
*bug 367533,
*opening the dropdown and not seeing the items (while they were actually there to use),
*having the en-US dictionnary at the end rather than at the beginning of the list.

In other words, I don't know if there is an issue with the current SM code and why you wanted that TB patch to be ported.
But if you (still) do, I would need some hints about how you want it done...