87028 - Improve wording about Turbo mode in Preferences

Status: VERIFIED FIXED

(SeaMonkey :: Preferences, defect)

Description

[doctor__j]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Win98; en-US; rv:0.9.1+) Gecko/20010620
BuildID:    2001062009

The current wording, "(reboot required)" is easily mistaken by user (or just me
only?!) that Mozilla will reboot the machine after closing Pref menu, just like
what usually happens after installing/uninstalling softwares.

Suggestion:  "(Effective after reboot)"
             "(Effective after you have rebooted)"

The suggested wording is intended to emphasize the fact that you *CAN* keep on
using the browser and there's no rush to reboot the system, or, to put it the
other way round, Mozilla will not FORCE a reboot.

Comment 1

[rbs]

Another suggestion?
-(Effective after reboot)
+(Effective after your next reboot)

Comment 2

[doctor__j]

Ah... "(Effective after your next reboot)" sounds very nice...
Cool... I'll vote for this one.

Comment 3

[sairuh (rarely reading bugmail)]

which prefs panel does the turbo stuff live in? couldn't find it in Advanced,
then again i'm using 2001.06.20.15 bits...

Comment 4

[sairuh (rarely reading bugmail)]

scratch that last comment --silly me, -turbo ain't in unix builds. :) anyhow, do
see it in today's win32 bits...

Comment 5

[Bradley Hart]

Taking, patch coming soon.

Comment 6

[Bradley Hart]

Attachment: Patch

Comment 7

[sairuh (rarely reading bugmail)]

r=sairuh, if you change it to "Enable Quick Launch (effective after restarting
your computer)"

Comment 8

[Bradley Hart]

Attachment: Patch that does exactly this

Comment 9

[Darin Fisher]

sr=darin

Comment 10

[[e-mail] (gone - use [e-mail] instead)]

Fix checked in:

C:\moz_src\mozilla\xpfe\components\prefwindow\resources\locale\en-US>cvs commit
-m "87028 - Improve wording about Turbo mode in Preferences.  Patch by Bradley H
art <[e-mail]> r=[e-mail] sr=[e-mail]" pref-advanc
ed.dtd
Checking in pref-advanced.dtd;
/cvsroot/mozilla/xpfe/components/prefwindow/resources/locale/en-US/pref-advanced
.dtd,v  <--  pref-advanced.dtd
new revision: 1.18; previous revision: 1.17
done

Comment 11

[sairuh (rarely reading bugmail)]

stephend, i presume your checkin was trunk-only? just a-wonderin'. is there a
huge call for this to get this into the branch as well? paul, lisa, your thoughts?

Comment 12

[[e-mail] (gone - use [e-mail] instead)]

Trunk only, for sure.

Comment 13

[lchiang]

Would be good for limbo on branch.  I'll mark nsBranch.  Stephen, can you check
into branch or find someone to do so when the time is appropriate?

Comment 14

[Bradley Hart]

I have e-mailed drivers asking for a= to check this into the 0.9.2 branch.

Comment 15

[Bradley Hart]

Asa informs me that 0.9.2 is no longer accepting checkins, so I guess this 
missed the branch :-(

Is it still possible to get this into a commercial branch?

Comment 16

[lchiang]

Yes, it is possible.   To be consistent in tracking this, I'm going to reopen
this bug and add "vtrunk" keyword.  This will indicate that the bug is fixed on
the trunk, but waiting for branch checkin after PDT approval.  (This all happens
after mozilla 0.9.2 milestone is out.)

Comment 17

[[e-mail] (gone - use [e-mail] instead)]

Bhuvan said he's check this in for me, thanks Bhuvan!

Comment 18

[Bradley Hart]

I have sent mail to [e-mail], asking for a= to check this in.

Comment 19

[msanz]

I cannot approve this at this time. Final kit went out on Friday and this is a
polishing fix. Please, don't check it into the branch, just the trunk.

Comment 20

[Bradley Hart]

oh, well.

Marking fixed since this is already in the trunk.

Comment 21

[Grace Bush]

verified fixed