410689 - Cannot change text color on Google documents

Status: RESOLVED DUPLICATE of bug 404433

(Core :: Widget: Cocoa, defect)

Description

[Håkan Waara]

Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9b3pre) Gecko/2008010304 Minefield/3.0b3pre

Steps to reproduce:

1. Create a new document
2. Write some text
3. Select the text, and try to change the color of it.

Expected result:

Color should be changed accordingly

Actual result:

No color change. Nothing happens.

Disclaimer:

I do not have time now to verify that the bug is in Gecko. There is a chance that it's Google doing something fishy; I'm reporting to be sure 1.9 does not ship, breaking some feature unknowingly.

Comment 1

[Ria Klaassen (not reading all bugmail)]

No problem with the latest trunk on Windows XP.

Comment 2

[Mats Palmgren (inactive)]

WFM, Firefox 2008010304 on Linux.
Confirming bug, Firefox 2008010304 on MacOSX.

I noticed there is a difference in how the selection is highlighted -
on Linux the selection is greyed out when clicking on the "color menu",
after clicking on a color [and the menu closes] it becomes blue
(primary selection) again.  On MacOSX, the selection stays grey after
choosing a color.  In Firefox 2.0.0.11 on MacOSX it becomes blue again.

The regression window is: 2006-09-28-08 -- 2006-09-29-06
http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=all&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=2006-09-28+07%3A00&maxdate=2006-09-29+07%3A00&cvsroot=%2Fcvsroot
I'm guessing bug 326469.

Comment 3

[Håkan Waara]

So apparently it's broken from the cocoa widgets switch.

Now that this is now confirmed, I suggest this should block 1.9 since Google documents is a pretty high-profile app.

Comment 4

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Isn't this bug 404433 (which was FIXED while the last patch on bug 403232 was checked in, but should be broken again since that was backed out)?

Comment 5

[Steven Michaud [:smichaud] (Retired)]

You're right, Smokey.

And this bug is also fixed by my patch(es) for bug 403232.