45360 - Pref: Browser/Editor Char Coding Default list should not mix Auto-detection modules and charset item

Status: VERIFIED FIXED

(Core :: Internationalization, defect, P3)

Description

[Katsuhiko Momoi]

** Observed with 7/12/2000 Win32 build **

Currently, the Browser/Editor Default Character Coding list
in Edit | Prefs | Navigator | Languages | Character Coding
includes both Auto-Detect moodules and real encoding items.

Since this pref item slipped in at the end, we have not had much
discussion about the specbut I think the current UI is incorrect: 

1. We should not be mixing Auto-Detect modules and encoding items
   when you can choose only 1 item. Conceptually, we now distinguish
   real encodings from Auto-Detect modules. They operate separately.
  
   I suggest that we create a separate Auto-Detect selection list.

2. In the Character Coding menu, selection of a real encoding
   item does not de-select an Auto-Detect module which has been 
   chosen. Yet in this Pref list, if you select an Auto-Detect module,
   you can't choose an encoding. If you choose an encoding, you
   can't choose an Auto-Detect module.

3. In shipping Mozilla to Japan, we should set 1) Auto-Detect
   AND 2) encoding default (Shift_JIS).

   For example, we want an explicit encoding name as the default 
   encoding for a new Editor document, NOT an Auto-Detect module.

Comment 1

[nhottanscp]

Reassign jbetak's bug to ftang.

Comment 2

[cata]

[ftang using cata's account] also, if we set it to some value, the value in 
pref.js is wrong. We should fix this for nsbeta2. This is blocking 45348. This 
part of code is very broken. We should fix this for nsbeta2 for sure. 

Comment 3

[Jay Patel [:jay]]

Putting on [nsbeta2-] radar.  Not critical to beta2.  Adding "relnote" keyword 
for PR2 release. 

Comment 4

[Frank Tang]

this bug have been work around by using MailCharetMenuRoot instead of 
BrowserCharsetMenuRoot . We should fix this for nsbeta3 

Comment 5

[Frank Tang]

cata's estimate is BEST:3 days/MOSTLIKELY: 4 days/ WORST: 5 days

Comment 6

[Frank Tang]

Marked as fix, remaining issues elsewhere.

Comment 7

[Teruko Kobayashi]

Auto-Detect modules are not the Default Character list.  I verified this in 
2000-08-25-08 build.