1292629 - One-click search engines: Don't loose input when leaving keyword input field with click on white list background.

Status: VERIFIED FIXED

(Firefox :: Settings UI, defect, P2)

Description

[Claas Augner [:claas]]

Firefox Desktop's preferences tab "Search" (about:preferences#search) includes a manageable list of "One-click search engines". For each search engine, a keyword can be chosen double-clicking the keyword cell of the engine's row or by alternatively pressing |F2| when the engine's row is selected.

Problem: Changing the keyword and then clicking on the white background of the engine list (below the last engine) looses the input, as if |Esc| had been pressed.

Solution: Persist keyword input when loosing focus by clicking on the white background.

Comment 1

[Claas Augner [:claas]]

Looks like we only loose the input when clicking in the "Search Engine" column.
When clicking in the "Keyword" column, the changed keyword is persisted as expected.
It looks like the `if` condition at [0] is the reason.

[0] https://dxr.mozilla.org/mozilla-central/rev/0ba72e8027cfcbcbf3426770ac264a7ade2af090/toolkit/content/widgets/tree.xml#403-404

Comment 2

[Claas Augner [:claas]]

Attachment: One-click search engines: Don't loose input when leaving keyword input field with click on white list background

Comment 3

[Claas Augner [:claas]]

Comment on attachment 8818376 [details] [diff] [review]
One-click search engines: Don't loose input when leaving keyword input field with click on white list background

Clicking on the white list background when editing a search engine's keyword triggers a blur event which discarded the input via |gSearchPane.onInputBlur| which used to call |tree.stopEditing(false)|.

I changed the behavior to accept the input unless it was discarded explicitly using the Escape key.

Comment 4

[Jared Wein [:jaws] (please needinfo? me)]

Comment on attachment 8818376 [details] [diff] [review]
One-click search engines: Don't loose input when leaving keyword input field with click on white list background

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

Looks good, thanks!

Please update your commit message to:
Bug 1292629 - Persist the keyword input field of the one-click search settings when the filed loses focus unless Escape is pressed. Previously the field would only persist if Enter was used. r=jaws"

Then re-upload the patch here and I will mark it for checkin-needed.

Comment 5

[Claas Augner [:claas]]

Attachment: Persist the keyword input field of the one-click search settings when the filed loses focus unless Escape is pressed. Previously the field would only persist if Enter was used

Comment 6

[Claas Augner [:claas]]

Comment on attachment 8818489 [details] [diff] [review]
Persist the keyword input field of the one-click search settings when the filed loses focus unless Escape is pressed. Previously the field would only persist if Enter was used

Thanks for the review, Jared. I have updated the commit message as requested.

Comment 7

[Jared Wein [:jaws] (please needinfo? me)]

Comment on attachment 8818489 [details] [diff] [review]
Persist the keyword input field of the one-click search settings when the filed loses focus unless Escape is pressed. Previously the field would only persist if Enter was used

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

Yep, looks good, and tested well.

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/25cf3827a16e
Persist the keyword input field of the one-click search settings when the filed loses focus unless Escape is pressed. Previously the field would only persist if Enter was used. r=jaws

Comment 9

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/25cf3827a16e

Comment 10

[Liz Henry (:lizzard) (relman/hg->git project)]

Want to request uplift to aurora for this fix?

Comment 11

[Jared Wein [:jaws] (please needinfo? me)]

Comment on attachment 8818489 [details] [diff] [review]
Persist the keyword input field of the one-click search settings when the filed loses focus unless Escape is pressed. Previously the field would only persist if Enter was used

Approval Request Comment
[Feature/Bug causing the regression]: introduced with new one-off search implementation, usability issue
[User impact if declined]: it is easy for users to make a change and then have that change get lost if they don't hit "Enter" after typing their keyword
[Is this code covered by automated tests?]: none, manual testing only. it only affects the Search pane of the Preferences
[Has the fix been verified in Nightly?]: manually by me but not by QA
[Needs manual test from QE? If yes, steps to reproduce]: no
[List of other uplifts needed for the feature/fix]: none
[Is the change risky?]: no
[Why is the change risky/not risky?]:
[String changes made/needed]:

Comment 12

[Liz Henry (:lizzard) (relman/hg->git project)]

Comment on attachment 8818489 [details] [diff] [review]
Persist the keyword input field of the one-click search settings when the filed loses focus unless Escape is pressed. Previously the field would only persist if Enter was used

Fix for minor usability regression in preferences pane, let's uplift to aurora for 52.

Comment 13

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/f05b1a13f410

Comment 14

[Mohammad Maruf Rahman]

I have reproduced this bug with Nightly 51.0a1 (2016-08-05) on WIndows 8.1 ,64 Bit!

This bug's fix is verified with latest Beta and Developer Edition (Aurora) ! 

Build    ID : 20170216105119
User  Agent : Mozilla/5.0 (Windows NT 6.3; Win64; x64; rv:52.0) Gecko/20100101 Firefox/52.0

Build    ID : 20170216004023
User  Agent : Mozilla/5.0 (Windows NT 6.3; Win64; x64; rv:53.0) Gecko/20100101 Firefox/53.0

[testday-20170217]

Comment 15

[Md. Majedul islam]

I have reproduced this bug with Nightly 51.0a1 (2016-08-05) on ubuntu 16.10,64 Bit!

This bug's fix is verified with latest Beta and Developer Edition (Aurora) ! 

Build    ID : 20170216105119
User  Agent : Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Firefox/52.0

Build    ID : 20170216084115
User  Agent : Mozilla/5.0 (X11; Linux x86_64; rv:53.0) Gecko/20100101 Firefox/53.0

[testday-20170217]

Comment 16

[Jared Wein [:jaws] (please needinfo? me)]

Thanks!