Ger Teunis Reporter
	Description • 19 years ago

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b2) Gecko/20050610 Firefox/1.0+
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b2) Gecko/20050610 Firefox/1.0+

When the Color is changed the "Set wallpaper" button doesn't work.

Reproducible: Always

Steps to Reproduce:
1. Right-click a image and select "Set as wallpaper"
2. Change the color
3. Click on "Set wallpaper" button

Actual Results:  
Nothing happens

Expected Results:  
The wallpaper should be set, and the "set as wallpaper" should disappear.

	Adam Guthrie
	Comment 1 • 19 years ago

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b2) Gecko/20050610
Firefox/1.0+ ID:2005061019

Confirming. This is definitely a regression.

	José Jeria
	Comment 2 • 19 years ago

Seing this as well, no js errors in js console

--> NEW

	José Jeria
	Comment 3 • 19 years ago

When did this regress?

	Ger Teunis Reporter
	Comment 4 • 19 years ago

(In reply to comment #3)
> When did this regress?

I checked back to 2005-05-30-05-trunk, all have the bug.
Fx 1.0 release did not have this bug.
Anyone knows a ftp with all/more nightly build?

	José Jeria
	Comment 5 • 19 years ago

(In reply to comment #4)
> Anyone knows a ftp with all/more nightly build?

http://archive.mozilla.org/

	Ger Teunis Reporter
	Comment 6 • 19 years ago

Thanks, it regressed between 20050224 and 20050225 build.
Not that great work tracing this regression down :-)

	Ger Teunis Reporter
	Comment 7 • 19 years ago

(In reply to comment #6)
> Thanks, it regressed between 20050224 and 20050225 build.
> Not that great work tracing this regression down :-)

Forgot to mention:
I've tested the win32 builds
(regressed between 2005-03-24-07-trunk and 2005-03-25-08-trunk)

	José Jeria
	Comment 8 • 19 years ago

Bugs checked in during that time period:
http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=PhoenixTinderbox&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=2005-03-24+07%3A00%3A00&maxdate=2005-03-25+08%3A00%3A00&cvsroot=%2Fcvsroot

	José Jeria
	Comment 9 • 19 years ago

cc:ing Mike Connor to see if he knows what bug might have caused this

	:Gavin Sharp [email: gavin@gavinsharp.com]
	Comment 10 • 19 years ago

This is a regression from Ben's pref dialog rewrite, specifically, the
colorpicker.xml changes.

http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&file=colorpicker.xml&branch=&root=/cvsroot&subdir=mozilla/toolkit/content/widgets&command=DIFF_FRAMESET&rev1=1.7&rev2=1.8

There's something wacky going on with generated events. Ben might not have
considered the use of colorpicker's other than those used by the PrefWindow.

	Asaf Romano (gone) Assignee
	Comment 11 • 19 years ago

-> me for investigation.

	Asaf Romano (gone) Assignee
	Comment 12 • 19 years ago

The onchange event in setDefaultBackground.xul reffers to |this.color|; however
the new |_fireEvent| method (in colorpicker.xml) doesn't fire the event
function with the right context object (which should be |aTarget|).

The attached patch also fixes a stupid js warning and the same error in the
preferences binding.

	Mike Connor [:mconnor]
	Comment 13 • 19 years ago

Comment on attachment 188251 [details] [diff] [review]
patch

please fix the bogus spacing in both places in the var rv = lines

	Asaf Romano (gone) Assignee
	Comment 14 • 19 years ago

Comment on attachment 188251 [details] [diff] [review]
patch

Might be too late for b3

	Asaf Romano (gone) Assignee
	Comment 15 • 19 years ago

and re-nominating...

	Asaf Romano (gone) Assignee
	Comment 16 • 19 years ago

In case this fix will be landed for b3/a2, the preferences dialog (the colors
preferences dialog in particular) should be tested.

	:Gavin Sharp [email: gavin@gavinsharp.com]
	Comment 17 • 19 years ago

There was bug 298665 filed for the actual widget work (see depends). Guess that
can be duped here now.

	Asaf Romano (gone) Assignee
	Comment 18 • 19 years ago

Checking in colorpicker.xml;
/cvsroot/mozilla/toolkit/content/widgets/colorpicker.xml,v  <--  colorpicker.xml
new revision: 1.10; previous revision: 1.9
done
Checking in preferences.xml;
/cvsroot/mozilla/toolkit/content/widgets/preferences.xml,v  <--  preferences.xml
new revision: 1.22; previous revision: 1.21
done