Serge Gautherie (:sgautherie) Assignee
	Comment 1 • 20 years ago

Diff from Xpfe to Toolkit version.

These came from Toolkit versions:
{{
1.11	ben%bengoodger.com	2005-02-25 01:07		274712 - new
options dialog
1.2	blakeross%telocity.com	2002-11-10 18:25		Fix the delay
when arrow keying up and down in the themes/extensions listboxes (and other
listboxes).
}}

	Serge Gautherie (:sgautherie) Assignee
	Comment 2 • 20 years ago

(In reply to comment #1)
> Created an attachment (id=176653) [edit]
> 2005.03.08 X2T diff report

This one would look simple enough for me to take:
What would be the expected non-regression test(s) for these changes ?

	Serge Gautherie (:sgautherie) Assignee
	Comment 3 • 20 years ago

(Jan: Could you update your address on <http://www.mozilla.org/owners.html> ?)

	neil@parkwaycc.co.uk
	Comment 4 • 20 years ago

Comment on attachment 176653 [details] [diff] [review]
2005.03.08 X2T diff report
[Checked in: Comment 6]

r+sr=me

The test would be to open e.g. Themes preferences and look how quickly it
responds to the arrow keys.

	Christian :Biesinger (don't email me, ping me on IRC)
	Comment 5 • 20 years ago

*** Bug 271935 has been marked as a duplicate of this bug. ***

	Serge Gautherie (:sgautherie) Assignee
	Comment 6 • 20 years ago

Comment on attachment 176653 [details] [diff] [review]
2005.03.08 X2T diff report
[Checked in: Comment 6]


Check in: { 2005-03-09 05:47	neil%parkwaycc.co.uk	mozilla/ xpfe/ global/
resources/ content/ bindings/ listbox.xml	 1.23 }

	Serge Gautherie (:sgautherie) Assignee
	Comment 7 • 20 years ago

(In reply to comment #4)
> (From update of attachment 176653 [details] [diff] [review] [edit])
> r+sr=me

(Neil, for the record, could you add these flags to the patch too ;->)

> The test would be to open e.g. Themes preferences and look how quickly it
> responds to the arrow keys.

I tried this and Languages (-> Add):
I can say it still works :-)
(then I could not tell the (VK_UP/VK_DOWN) speed difference ... anyway.)

	Christian :Biesinger (don't email me, ping me on IRC)
	Comment 8 • 20 years ago

serge, it was your patch :) no reason to assign to me.

	Serge Gautherie (:sgautherie) Assignee
	Comment 9 • 19 years ago

A few space nits,
and a |var| redeclaration.

	Serge Gautherie (:sgautherie) Assignee
	Comment 10 • 19 years ago

Comment on attachment 210717 [details] [diff] [review]
(Bv1) nits, in the meantime
[Checked in: Comment 10]


Check in: { 2006-02-24 07:16	bugzilla%standard8.demon.co.uk }

	Robert Kaiser
	Comment 11 • 17 years ago

SeaMonkey is now using toolkit, do we still need to port anything from xpfe's listbox.xml to toolkit or can we close this bug?

	neil@parkwaycc.co.uk
	Comment 12 • 17 years ago

No, the only bonus xpfe listbox has over toolkit is that it has none of the regressions caused by bug 281053 (which are few and far between for listboxes).

	Robert Kaiser
	Comment 13 • 17 years ago

in this case, I'm re-marking the bug fixed (due to it containing fixes that have been checked in). Please reopen or file new bugs if we run across something unexpected we still would need to port to toolkit.