Retained results: If you start to search, then click outside the bar and back in, the results should be shown again
Categories
(Firefox :: Address Bar, enhancement, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr68 | --- | unaffected |
firefox71 | --- | disabled |
firefox72 | --- | disabled |
firefox73 | --- | disabled |
firefox74 | --- | verified |
People
(Reporter: dao, Assigned: mak)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
Updated•5 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
So pretty much this is openViewOnFocus, with a couple changes.
The first difference is that we should still open the view in the newtab page, provided there is a search string. This is compatible with the current behavior for topsites since it doesn't overlap it.
The second difference is relative to private browsing, we don't open the view for top sites in PB mode, it's unclear whether we should do the same in this case, so I'm asking Verdi on Slack.
Reporter | ||
Comment 2•5 years ago
|
||
(In reply to Marco Bonardo [::mak] from comment #1)
So pretty much this is openViewOnFocus, with a couple changes.
Yeah, I think it should also depend on that mode actually being enabled. I'm not sure what the plan is there beyond the top sites experiment; are we considering enabling open-view-on-focus in 71?
Assignee | ||
Comment 3•5 years ago
|
||
Verdi clarified it should work the same in PB mode.
I was problably wrong about the overlapping though, I can't just flip the pref.
If there's a search string and the megabar is enabled, we can surely open the view.
But, if there's no search string, we should open the view only on request (like for the topsites experiment).
I can likely reuse the same hook points, just not the openViewOnFocus pref.
Assignee | ||
Comment 4•5 years ago
|
||
(In reply to Dão Gottwald [::dao] from comment #2)
Yeah, I think it should also depend on that mode actually being enabled. I'm not sure what the plan is there beyond the top sites experiment; are we considering enabling open-view-on-focus in 71?
I think openViewOnFocus will pretty much relate to the empty-string case and we'll likely flip that pref once we decide what to show in that case (top sites? something else). This depends on the experiments result.
Assignee | ||
Comment 5•5 years ago
|
||
Assignee | ||
Updated•5 years ago
|
Comment 7•5 years ago
|
||
bugherder |
Comment 8•5 years ago
|
||
I reproduced this issue using Fx 71.0a1 (2019-09-05).
I can confirm this issue is fixed, I verified using Fx 74.0a1 (2020-02-04), on Windows 10 x64, Ubuntu 18.04 and macOS 10.15.
Updating the flags accordingly.
Description
•