View / Character Encoding Menu is Missing on the Trunk

RESOLVED FIXED

Status

Thunderbird
Mail Window Front End
RESOLVED FIXED
11 years ago
11 years ago

People

(Reporter: Scott MacGregor, Assigned: philor)

Tracking

({regression})

x86
Windows Vista
regression

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

11 years ago
Using a trunk build from today, David and I just noticed that the View / Character encoding menu is missing.
(Assignee)

Comment 1

11 years ago
Created attachment 276562 [details] [diff] [review]
Fix v.1

Regression range:
http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=all&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=2007-07-14+16%3A18%3A00&maxdate=2007-07-14+16%3A20%3A00&cvsroot=%2Fcvsroot

Sure, it was an effective way of dealing with the unremoved mailCharsetLoadListener, but maybe a little too extreme?
Assignee: nobody → philringnalda
Status: NEW → ASSIGNED
Attachment #276562 - Flags: review?(mscott)
(Reporter)

Comment 2

11 years ago
Comment on attachment 276562 [details] [diff] [review]
Fix v.1

ah two beers in one day. that charset overlay removal was me messing around with something else and shouldn't have been checked in anyway.

Did you mean to switch to tookit's printUtils.js file too?
Attachment #276562 - Flags: review?(mscott) → review+
(Assignee)

Comment 3

11 years ago
Heh. No, that was me not looking at my patch, that's still waiting on bug 321954 / bug 387247. With a lot of luck, the tree will reopen before I forget that I need to not break printing while fixing encoding.
(Assignee)

Comment 4

11 years ago
mail/base/content/mailWindowOverlay.xul 1.218
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.