Steffen Wilberg Assignee
	Description • 9 years ago

In the in-content preferences, Applications pane, the keys to focus the Search box (Ctrl/Cmd+f and Ctrl/Cmd+k) are broken.
Accel+f now invokes the find bar instead, which is useless.

This is a regression from bug 1016300, which removed the oncommand attribute from the <key> elements. This doesn't work because of bug 371900.

The fix is to add |oncommand=";"| to the <key> elements.
Bug 978115 is on file to clean up these workarounds once bug 371900 is fixed.

	:Gijs (he/him)
	Comment 2 • 9 years ago

Comment on attachment 8645435 [details] [diff] [review]
In the Applications in-content preferences, fix the Ctrl/Cmd+f and Ctrl/Cmd+k keys to focus the search box

Review of attachment 8645435 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks for the super-clear explanation in comment #0. Wish all bugs came with descriptions that good.

Maybe leave a code comment (<!-- ... -->) before these two <key>s so we don't accidentally remove them again - something like "oncommand attributes are here because of bug 371900"?

	Pulsebot
	Comment 3 • 9 years ago

https://hg.mozilla.org/integration/fx-team/rev/aac05e8ce3ec

	:Gijs (he/him)
	Comment 4 • 9 years ago

(In reply to :Gijs Kruitbosch from comment #2)
> Maybe leave a code comment (<!-- ... -->) before these two <key>s so we
> don't accidentally remove them again - something like "oncommand attributes
> are here because of bug 371900"?

(In reply to Pulsebot from comment #3)
> https://hg.mozilla.org/integration/fx-team/rev/aac05e8ce3ec

I should really not land things without having tea in the morning.

	Pulsebot
	Comment 5 • 9 years ago

https://hg.mozilla.org/integration/fx-team/rev/6509f5665152

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 6 • 9 years ago

https://hg.mozilla.org/mozilla-central/rev/aac05e8ce3ec
https://hg.mozilla.org/mozilla-central/rev/6509f5665152

	Khalid Syfullah Zaman [:khalid32]
	Comment 7 • 9 years ago

I have reproduced this bug on Nightly 42.0a1 (2015-08-09) on ubuntu 14.04 LTS, 32 bit!

The bug's fix is now verified on Latest Beta 43.0b3!

Build ID: 20151112144305
User Agent: Mozilla/5.0 (X11; Linux i686; rv:43.0) Gecko/20100101 Firefox/43.0

[testday-20151113]