Closed Bug 31136 Opened 20 years ago Closed 19 years ago

Rename prefs from "browser.xxx" to "browser.display.xxx"

Categories

(SeaMonkey :: Preferences, defect, P3)

defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: pierre, Assigned: pierre)

Details

(Whiteboard: [nsbeta2+])

Attachments

(2 files)

This bug is extracted from bug 29304.
The following prefs:
    browser.background_color
    browser.foreground_color
    browser.use_document_fonts
    browser.use_document_colors
    browser.base_font_scaler
    browser.wfe.use_windows_colors
    browser.screen_resolution
should be renamed as "browser.display.xxx" instead of "browser.xxx".
Changing component to Preferences
Assignee: neeti → matt
Component: Preferences: Backend → Preferences
Reassigning as per Don
Assignee: matt → mcafee
Target Milestone: --- → M18
I have this change ready to go.
Status: NEW → ASSIGNED
fixed.
QA: try setting some colors in prefs and make sure they
have a "browser.display" prefix.  Oh yeah, color pref window
is busted so ? might need to hand-test.  Or you can just take
my word for it that the changes went in.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
yep, they're working --testing on linux [opt comm, 2000.05.18.08] with these
prefs:

browser.display.background_color
browser.display.screen_resolution
browser.display.use_document_fonts
Status: RESOLVED → VERIFIED
Sorry, I was on vacation last week and did not have an opportunity to review the 
code until now. I'm reopening the bug for a small but important code change. The 
goal of this bug (and the result of the long discussion we had in bug 29304) was 
to remove all the RegisterCallback()s from nsPresContext.cpp and replace them 
with a single:
      RegisterCallback("browser.display.", PrefChangedCallback, (void*)this);
and then ask people who want a new pref to cause a reflow when it changes to call 
it "browser.display.<something>".

Chris, it's a 2-line change. Feel free to reassign to me if you want.
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
I did the dirty work, looks like we need to turn it on now.
pierre, can you drive this one?
Assignee: mcafee → pierre
Status: REOPENED → NEW
Status: NEW → ASSIGNED
Marking nsbeta2/fix in hand. Very simple fix...
Keywords: nsbeta2
Whiteboard: [fix in hand]
Putting on [nsbeta2+] radar.  Patch appears low risk. Approved to checkin for 
patch as shown.
Whiteboard: [fix in hand] → [nsbeta2+]
Fix checked in nsPresContext.cpp
Status: ASSIGNED → RESOLVED
Closed: 20 years ago19 years ago
Resolution: --- → FIXED
rubber-stamp vrfy.
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.