Closed
Bug 1055771
Opened 7 years ago
Closed 7 years ago
Search suggestions for old query appear when activity is re-launched
Categories
(Firefox for Android Graveyard :: Search Activity, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 34
People
(Reporter: Margaret, Assigned: Margaret)
References
Details
Attachments
(1 file, 1 obsolete file)
1.53 KB,
patch
|
wesj
:
review+
|
Details | Diff | Splinter Review |
Maybe this is a regression from bug 1052563.
Assignee | ||
Comment 1•7 years ago
|
||
We don't update suggestions when the text changes if the search bar isn't active. So in order to reset the suggestions we need to make sure to activate editing mode before resetting the query.
Attachment #8475514 -
Flags: review?(wjohnston)
Comment 2•7 years ago
|
||
Comment on attachment 8475514 [details] [diff] [review] Reset search suggestions when activity is launched from a new intent Review of attachment 8475514 [details] [diff] [review]: ----------------------------------------------------------------- I would probably make that comment a little more strongly worded. i.e. Something like "Suggestions won't update if we're not in edit mode." makes it clearer to me than "Reset the query after entering edit mode to update the suggestions appropriately."
Assignee | ||
Comment 3•7 years ago
|
||
(In reply to Wesley Johnston (:wesj) from comment #2) > Comment on attachment 8475514 [details] [diff] [review] > Reset search suggestions when activity is launched from a new intent > > Review of attachment 8475514 [details] [diff] [review]: > ----------------------------------------------------------------- > > I would probably make that comment a little more strongly worded. i.e. > Something like "Suggestions won't update if we're not in edit mode." makes > it clearer to me than "Reset the query after entering edit mode to update > the suggestions appropriately." Yeah, I don't love the way things like this are coupled right now :( I also thought about making an explicit clearSuggestions method on SuggestionsFragment. Maybe that would be a better solution.
Assignee | ||
Comment 4•7 years ago
|
||
Updated the comment to be more explicit about the ordering. Also, I looked into the logic for SuggestClient, and we don't perform a network request when there's an empty query; we just return an empty suggestion list. So I don't feel as bad about making this flow through the loadSuggestions code path.
Attachment #8475514 -
Attachment is obsolete: true
Attachment #8475514 -
Flags: review?(wjohnston)
Attachment #8476961 -
Flags: review?(wjohnston)
Updated•7 years ago
|
Attachment #8476961 -
Flags: review?(wjohnston) → review+
Assignee | ||
Comment 5•7 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/0654c821911a https://github.com/mozilla/fennec-search/commit/b41d8feed71873960c8ac389fead9e2f66c7969f
Comment 6•7 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/0654c821911a
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 34
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•