1136139 - [implement] remove suggestions area in search panel when suggestions are disabled or unavailable

Status: VERIFIED FIXED

(Firefox :: Search, defect, P3)

Description

[Ash Grigas]

Build ticket for https://bugzilla.mozilla.org/show_bug.cgi?id=1109636

See attachment on bug 1109636 for UI change.

If user has turned off suggestions or suggestions are not available from their default provider, remove the white space and duplicate search string shown currently.

If history item exists for search string, it should still show even if suggest is off.

Comment 1

[Florian Quèze [:florian]]

(In reply to agrigas from comment #0)

> If user has turned off suggestions or suggestions are not available from
> their default provider, remove the white space and duplicate search string
> shown currently.
> 
> If history item exists for search string, it should still show even if
> suggest is off.

This just means revert this change: http://hg.mozilla.org/mozilla-central/diff/495a9f216f11/toolkit/components/search/nsSearchSuggestions.js

Comment 2

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

(In reply to Florian Quèze [:florian] [:flo] (Away until March 11) from comment #1)
> This just means revert this change:
> http://hg.mozilla.org/mozilla-central/diff/495a9f216f11/toolkit/components/
> search/nsSearchSuggestions.js

Is another change needed to ensure that the panel opens anyways, or is that comment just wrong now?

Comment 3

[Florian Quèze [:florian]]

(In reply to :Gavin Sharp [email: [e-mail]] from comment #2)
> (In reply to Florian Quèze [:florian] [:flo] (Away until March 11) from
> comment #1)
> > This just means revert this change:
> > http://hg.mozilla.org/mozilla-central/diff/495a9f216f11/toolkit/components/
> > search/nsSearchSuggestions.js
> 
> Is another change needed to ensure that the panel opens anyways, or is that
> comment just wrong now?

Indeed, there's probably another change needed to force-open the panel.

Comment 4

[:Gijs (he/him)]

remote:   https://treeherder.mozilla.org/#/jobs?repo=try&revision=0813214c0d13

Comment 5

[:Gijs (he/him)]

Attachment: MozReview Request: bz://1136139/Gijs

/r/7855 - Bug 1136139 - fix collapsing of search suggestions when there are none, r?florian

Pull down this commit:

hg pull -r 3965a73943aa40f0e1589f32b01e0224716062bb https://reviewboard-hg.mozilla.org/gecko/

Comment 6

[:Gijs (he/him)]

Comment on attachment 8599327 [details]
MozReview Request: bz://1136139/Gijs

/r/7855 - Bug 1136139 - fix collapsing of search suggestions when there are none, r?florian

Pull down this commit:

hg pull -r 3965a73943aa40f0e1589f32b01e0224716062bb https://reviewboard-hg.mozilla.org/gecko/

Comment 7

[:Gijs (he/him)]

Comment on attachment 8599327 [details]
MozReview Request: bz://1136139/Gijs

/r/7855 - Bug 1136139 - fix collapsing of search suggestions when there are none, r?florian

Pull down this commit:

hg pull -r 46f43de7fe0123dab166a144db4620a93c080284 https://reviewboard-hg.mozilla.org/gecko/

Comment 8

[:Gijs (he/him)]

remote:   https://treeherder.mozilla.org/#/jobs?repo=try&revision=601fdc85e5e0

Comment 9

[Florian Quèze [:florian]]

Comment on attachment 8599327 [details]
MozReview Request: bz://1136139/Gijs

https://reviewboard.mozilla.org/r/7853/#review7059

Ship It!

Comment 10

[Ryan VanderMeulen [:RyanVM]]

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

Comment 11

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/e73e95394f91

Comment 12

[Petruta Horea [:phorea], Desktop Test Engineering]

I have verified this using Nightly 41.0a1 2015-05-26 under Win 7 64-bit, Mac OS X 10.9.5 and Ubuntu 12.04 32-bit.
I've found and logged bug 1169220 so I'm marking this one as verified.

Comment 14

[:Gijs (he/him)]

Attachment: MozReview Request: Bug 1136139 - fix collapsing of search suggestions when there are none, r?florian