335306 - Turn inline spellchecking on by default

Status: RESOLVED FIXED

(Firefox :: General, defect, P1)

Description

[Brett Wilson]

Comment 1

[Nick Thomas [:nthomas] (UTC+12)]

Looks like attachment 205238 [details] [diff] [review] from bug 319317 will need to land on the branch to ensure files are packaged properly.

Is this a meta bug, or will the patch to configure to modify the default extensions list appear here ?

Comment 2

[Brett Wilson]

(In reply to comment #1)
> Is this a meta bug, or will the patch to configure to modify the default
> extensions list appear here ?

This is where the patch will be. If somebody wants to make it, please go ahead. Otherwise I'll do it when I get everything else in.

I requested branch approval for bug 319317. Thanks.

Comment 3

[Brett Wilson]

Attachment: Patch

Comment 4

[Ben Goodger (use ben at mozilla dot org for email)]

Comment on attachment 220712 [details] [diff] [review]
Patch

r=[e-mail]

go go gadget spellcheck

Comment 5

[Brett Wilson]

Fixed on branch. Leaving open so I remember to do it on the trunk as well.

Comment 6

[pal-moz]

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8) Gecko/20060503 BonEcho/2.0a1 ID:2006050317

if focus is not on text area, the line disappear at random.
is this related to this bug/checkin?

screen shot : http://img79.imageshack.us/img79/3540/spellcheck4rc.jpg

Comment 7

[Brett Wilson]

(In reply to comment #6)
> if focus is not on text area, the line disappear at random.
> is this related to this bug/checkin?

I have no idea. Can you file another bug?

Comment 8

[pal-moz]

(In reply to comment #7)
> (In reply to comment #6)
> > if focus is not on text area, the line disappear at random.
> > is this related to this bug/checkin?
> 
> I have no idea. Can you file another bug?
> 

filed a bug.
https://bugzilla.mozilla.org/show_bug.cgi?id=336679

Comment 9

[Nick Thomas [:nthomas] (UTC+12)]

Attachment: Trunk patch

Comment 10

[Brett Wilson]

Fixed on trunk.