Clicking "Ask again" just appends--rather than replaces--article search results from the AAQ form.

VERIFIED FIXED in 1.5.1

Status

--
major
VERIFIED FIXED
9 years ago
9 years ago

People

(Reporter: stephend, Assigned: paulc)

Tracking

unspecified
1.5.1

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(2 attachments)

(Reporter)

Description

9 years ago
Summary: Clicking "Ask again" just appends--rather than replaces--article search results from the AAQ form.

STR:

1. Load http://support-stage.mozilla.org/en-US/forum/1?openpost=1
2. Choose the last radio button and click "Next step"
3. On the next screen, click "Next step" again
4. Now, type anything ("Google") and click "Ask this"
5. Click "Ask again"

Expected Results:

We should just replace the set of results, IMHO (even if the search term changes), rather than appending

Actual Results:

For the same search term, we just keep appending each set of results
(Reporter)

Updated

9 years ago
Blocks: 537781
(Reporter)

Comment 1

9 years ago
Created attachment 420668 [details]
Screenshot
/agree with the expected result. Every search should clear and repopulate the results (unless, I suppose, the query doesn't change, then it doesn't necessarily need to go back, but it's probably easiest to do it every time).
Assignee: nobody → paulc
(Assignee)

Comment 3

9 years ago
Created attachment 420780 [details] [diff] [review]
clear contents before appending

Wow. How did I miss this? Thank you Stephen.
Attachment #420780 - Flags: review?(james)
Comment on attachment 420780 [details] [diff] [review]
clear contents before appending

Works great.
Attachment #420780 - Flags: review?(james) → review+
(Assignee)

Comment 5

9 years ago
r59655
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
(Reporter)

Comment 6

9 years ago
Verified FIXED; thanks!

(I think we should expose the fact that the minimum search-query length is 5 characters, but that's a separate bug.)
Status: RESOLVED → VERIFIED
(Assignee)

Comment 7

9 years ago
Stephen: if you could, file that and assign it to me, by all means.
(Reporter)

Comment 8

9 years ago
(In reply to comment #7)
> Stephen: if you could, file that and assign it to me, by all means.

Bug 539313, thanks!
Status: VERIFIED → RESOLVED
Last Resolved: 9 years ago9 years ago
Reopening to commit to branches/1.5.1_temp. Will re-resolve after patches are committed.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
On branch, r62441
Status: REOPENED → RESOLVED
Last Resolved: 9 years ago9 years ago
Resolution: --- → FIXED
Verified, FIXED.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.