Closed Bug 1132351 Opened 9 years ago Closed 9 years ago

[Search] Loading animation loop does not disappear after all results are loaded

Categories

(Firefox OS Graveyard :: Gaia::Search, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(blocking-b2g:2.2+, b2g-v2.2 verified, b2g-master unaffected)

VERIFIED FIXED
2.2 S6 (20feb)
blocking-b2g 2.2+
Tracking Status
b2g-v2.2 --- verified
b2g-master --- unaffected

People

(Reporter: hcheng, Assigned: daleharvey)

References

Details

(Whiteboard: [systemsfe])

Attachments

(2 files, 1 obsolete file)

*Description:
The loading animation loop of search app does not disappear in below two situations.
 a. input some characters and then all results are loaded.
 b. after pressing "close" button, it is empty state.

*Env info
Build ID               20150211162505
Gaia Revision          791e53728cd8018f1d7cf7efe06bbeb1179f0370
Gaia Date              2015-02-11 19:31:02
Gecko Revision         https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/dcf030ef499d
Gecko Version          37.0a2
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150211.194909
Firmware Date          Wed Feb 11 19:49:19 EST 2015
Bootloader             L1TC000118D0
Attached image loading animation loop.jpg (obsolete) —
Besides, the place of the loading loop is not in the central.
Blocks: 1098494
blocking-b2g: --- → 2.2?
Whiteboard: [systemsfe]
This looks like a merge conflict resolved wrong, there are 2 loading indicators, one displays correct, the old one doesnt
Assignee: nobody → dale
Only appears on 2.2, not on master
Comment on attachment 8563288 [details] [review]
[gaia] daleharvey:1132351 > mozilla-b2g:v2.2

Most of the 2.2 uplifts have hit merge conflicts, this deletion managed to survive one
Attachment #8563288 - Flags: review?(bfrancis)
https://github.com/mozilla-b2g/gaia/commit/0f98995c5890232dd01ee3d17183937caec54337 is where is got introduced, that was supposed to move it not add another
Comment on attachment 8563288 [details] [review]
[gaia] daleharvey:1132351 > mozilla-b2g:v2.2

[Approval Request Comment]
[Bug caused by] (feature/regressing bug #): Merge conflict
[User impact] if declined: Redundant progress indicator
[Testing completed]:
[Risk to taking this patch] (and alternatives if risky):
[String changes made]: n/a
Attachment #8563288 - Flags: review?(bfrancis)
Attachment #8563288 - Flags: review+
Attachment #8563288 - Flags: approval-gaia-v2.2?
blocking-b2g: 2.2? → 2.2+
Attachment #8563288 - Flags: approval-gaia-v2.2? → approval-gaia-v2.2+
v2.2: https://github.com/mozilla-b2g/gaia/commit/7265c101501d65178aa033468792deadfd2c8524
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.2 S6 (20feb)
The place of loading loop is correct but the loading loop still keeps there.
Attachment #8563235 - Attachment is obsolete: true
Besides, after I click "x" button to clean the result, the loading loop does not disappear.

It also happens on master.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Bug 113419 results in the loading loop keeping there after input. Then, the loop does not disappear after clearing is from Bug 1131408.

Close this bug again and leave two issues to these 2 bugs.
Status: REOPENED → RESOLVED
Closed: 9 years ago9 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: