Closed Bug 123127 Opened 23 years ago Closed 23 years ago

"pref.browser.smartbrowsing.disable_textbox.add" should be "browser.smartbrowsing.disable_textbox.add"

Categories

(SeaMonkey :: Preferences, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED WONTFIX

People

(Reporter: bugzilla, Assigned: samir_bugzilla)

Details

currently there's a pref called: user_pref("pref.browser.smartbrowsing.disable_textbox.add", false); there's absolutely no need to include the "pref" word into a prefs...! The pref should be renamed to: user_pref("browser.smartbrowsing.disable_textbox.add", false); This way the pref will also be located next the the other browser prefs in the prefs.js file. 20020201 btw: I'm CC'ing the people that seems to be contributers to the file where the pref is used: http://lxr.mozilla.org/seamonkey/source/xpfe/components/prefwindow/resources/content/pref-smart_browsing.xul#70 NOTE: Is the pref actually used???? http://lxr.mozilla.org/seamonkey/search?string=disable_textbox seems to only get one hit!
How do you propose we handle backward compatibility? More code to convert older profiles? That seems like code bloat for a name we would like to have aligned with others. Is there some end user benefit?
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → WONTFIX
v
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.