Closed Bug 38318 Opened 24 years ago Closed 24 years ago

need to update fonts when prefs change

Categories

(Core :: Layout, defect, P3)

PowerPC
Mac System 8.0
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: erik, Assigned: sfraser_bugs)

References

Details

(Keywords: platform-parity, Whiteboard: [nsbeta2+])

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
Keywords: nsbeta2
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.
Whiteboard: [nsbeta2+]
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 garywade@desisoftsystems.com
Assignee: ftang → garywade
Status: ASSIGNED → NEW
Blocks: 24343
*** Bug 40219 has been marked as a duplicate of this bug. ***
spam: adding pp kw, adding self to cc list.
Keywords: pp
reassign sfraser.
sfraser- if this a dup of your fix yesterday, please mark it fixed. Otherwise, 
reassign this back to  garywade@desisoftsystems.com
Assignee: garywade → sfraser
This seems to work now.
Status: NEW → RESOLVED
Closed: 24 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.