1047688 - implement UI for nsIBrowserSearchService.selectEngine

Status: RESOLVED WONTFIX

(Firefox :: Search, defect, P4)

Description

[Drew Willcoxon :adw]

See breakdown bug 1029725.  Quoting https://wiki.mozilla.org/User:Mconnor/Search_API_Change:

> Follows the Win7 style wizard model of making a user choose an option. Cancel
> means nothing changes. Choosing current means the add-on can't re-prompt with
> the same version again. Ideally this should use the larger images introduced for
> new tab search, falling back to 32x32 or 16x16 icons if unavailable.
> 
> +-------------------------------------------------------+
> | Change Search Provider?                               |
> +-------------------------------------------------------+
> |                                                       |
> | Bob's Super Happy Toolbar wants you to change your    |
> | search provider to Ask.com   _Learn More_             |
> |                                                       |
> | [ [ Google ] Keep using Google (currently in use)   ] |
> |                                                       |
> | [ [Ask.com ] Change my search to Ask.com            ] |
> |                                                       |
> +-------------------------------------------------------+
> | (?)                                          [Cancel] |
> +-------------------------------------------------------+
> 
> * Users must make a positive choice to select the engine (no next button)
> * Source of the request must be identified
> * We should use the biggest images that make sense in context.
> * Learn more link should point to SUMO

I think this UI should be presented by an XPCOM component so that the search service can call it, as described in bug 1047687.

Comment 2

[:Gavin Sharp [email: [e-mail]]]

Mockup is in bug 1062525 comment 0.

Comment 3

[Mark Hammond [:markh] [:mhammond]]

I'm not going to get to this in this iteration.

Comment 5

[(Currently slow to respond) Philipp Sackl [:phlsa] (Firefox UX) please use needinfo]

For reference, a mockup of the updated UI can be found here: https://bugzilla.mozilla.org/attachment.cgi?id=8481471

Comment 6

[Florian Quèze [:florian]]

I'm tempted to wontfix this, but will wait on an answer to my needinfo in bug 1047687 first.

Comment 7

[Florian Quèze [:florian]]

(In reply to Florian Quèze [:florian] from comment #6)
> I'm tempted to wontfix this, but will wait on an answer to my needinfo in
> bug 1047687 first.

I got tired of waiting and wontfixed bug 1047687, so wontfix here too!