When the user changes the fonts in the prefs panel, Mozilla should update the fonts accordingly, but Mac gfx doesn't, yet. Frank, I don't know if there will be a timing problem. The layout engine already has a pref callback for the fonts, and it asks the device context to delete all of them before reflowing all documents. If this happens before you do whatever is needed in Mac gfx, then there may be a problem. (This problem does not occur on Windows and Unix since the pref-sensitive data is stored in nsIFontMetrics objects, which are deleted ("Flush").) Please keep this in mind when you work on this.
I think this should be nsbeta2.
Status: NEW → ASSIGNED
Target Milestone: --- → M17
move to M16
OS: Mac System 7 → Mac System 8.0
Target Milestone: M17 → M16
Putting on [nsbeta2+] radar for beta2 fix.
Running the daily build from Friday on the Mac, the problem seems to be that the Mac IGNORES COMPLETELY the font prefs. You can set whatever you want, quit, relaunch, verify that the prefs are still set but the text stays in Times/ Helvetica. The font size setting works though...
Partially fix the problem. Now the turkish / baltic font pref setting won't overrule the smRoman font pref. We will rebuild the mapping util singleton when the font pref changed. However, there are still problem. It is hard to fix now since the preference window is pretty bad now. After I open pref window and OK, the second time I open the prefwindow will cause problem and need quit.
Part of this problem on the Mac side may be related to the dpi pref. When I changed the value on the font preference window from 96 to 72 dpi (to better fit the Mac), the problem seemed to get worse. M15 did not seem to like it, as did today's daily Mac build (2000051908). The daily Mac build from today -- 2000051908 -- seemed much more unstable on an iMac DV. It wouldn't run for more than a couple minutes before crashing. FWIW, I'll check this again when a more stable M16 comes out. Oh, yes. Mac OS 9.0.4, MRJ 2.2, carbon libs 1.0.4, Apple multimedia update 1.0 installed.
reassign to email@example.com
Assignee: ftang → garywade
Status: ASSIGNED → NEW
*** Bug 40219 has been marked as a duplicate of this bug. ***
spam: adding pp kw, adding self to cc list.
reassign sfraser. sfraser- if this a dup of your fix yesterday, please mark it fixed. Otherwise, reassign this back to firstname.lastname@example.org
Assignee: garywade → sfraser
This seems to work now.
Status: NEW → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
Fixed in the June 6th Mac build (2000060608).
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.