Closed Bug 1042189 Opened 7 years ago Closed 7 years ago

Implement interaction designs for selecting a suggestion/history item

Categories

(Firefox for Android Graveyard :: Search Activity, defect, P1)

x86
Android
defect

Tracking

(firefox35 verified)

VERIFIED FIXED
Firefox 34
Tracking Status
firefox35 --- verified

People

(Reporter: antlam, Assigned: Margaret)

References

Details

Attachments

(3 files, 2 obsolete files)

Attached video search_suggexpand2.mov (obsolete) —
Opening up this bug to track the interactive design and animations work for the suggestions step in the search flow.

In particular, how interactions park will work when a user selects a suggestion/history item. Then, they'd be presented with the results page (web view for now).
Attaching screenshot just for quick reference.
Assignee: nobody → margaret.leibovic
Lucas, any suggestions about this best way to approach this?
Flags: needinfo?(lucasr.at.mozilla)
(In reply to :Margaret Leibovic from comment #2)
> Lucas, any suggestions about this best way to approach this?

I'd probably keep a top level TextView that could get positioned to overlap with the TextView in the clicked search suggestion row and then animated to the final position in the input entry. Then hide the animated TextView and set the input entry's text with the new content. Give this a try to see how the code would look like.
Flags: needinfo?(lucasr.at.mozilla)
Priority: -- → P1
Margaret made a great catch, the starting y position was way below the actual label so this video has that updated.
Attachment #8460369 - Attachment is obsolete: true
Depends on: 1044133
Here's a WIP PR:
https://github.com/ericedens/FirefoxSearch/pull/32

And here's a build to try:
http://people.mozilla.org/~mleibovic/search-anim.apk

Adding these transitions challenge our current modular fragment design, since this transition could be initiated from different fragments. I just stuck the logic in MainActivity for now, but I had to hack some public APIs into SearchFragment, so we should think about how we might want to update the interfaces between these different fragments.
Things to address in other bugs:

* Fine-tuning interpolator
* Search bar animation (we can do this in bug 1043027)
* Reverse animation when hitting back button (I'm not sure if we can even do this as indicated in the mock-up, since we would have to store where the original suggestion was, and we have to count on that suggestion still being in that same spot)
* Refactoring interfaces to make transition logic less fragile

Once we land this patch, we can divide and conquer on the follow-up issues.
Attachment #8464133 - Flags: review?(eric.edens)
Comment on attachment 8464133 [details] [diff] [review]
Animation for selecting a suggestion/history item

Review of attachment 8464133 [details] [diff] [review]:
-----------------------------------------------------------------

This looks great!

We talked about this on IRC, but for bugzilla: I tried this on Gingerbread S2, and the animations look nice. At the end of the animation, though, the webview is hidden -- perhaps the animationText and/or the animationCard are still visible? So we should probably figure that out before landing. (Whether disabling animations in Gingerbread, or working around whatever is causing the issue)

::: mobile/android/search/java/org/mozilla/search/AcceptsSearchQuery.java
@@ +28,5 @@
> +    /**
> +     * Interface to specify search suggestion animation details.
> +     */
> +    public interface SuggestionAnimation {
> +        public Rect getStartBounds();

I like this SuggestionAnimation interface :)

::: mobile/android/search/java/org/mozilla/search/MainActivity.java
@@ +170,5 @@
> +            }
> +        });
> +
> +        set.setDuration(SUGGESTION_TRANSITION_MS);
> +        set.setInterpolator(new AccelerateDecelerateInterpolator());

Can AccelerateDecelerateInterpolator be cached in a static final?
Attachment #8464133 - Flags: review?(eric.edens) → feedback+
This fixes the gingerbread issues. I built this on top of my patch for bug 1045819.
Attachment #8464133 - Attachment is obsolete: true
Attachment #8464253 - Flags: review?(eric.edens)
Comment on attachment 8464253 [details] [diff] [review]
(v2) Animation for selecting a suggestion/history item

Review of attachment 8464253 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good!
Attachment #8464253 - Flags: review?(eric.edens) → review+
https://hg.mozilla.org/mozilla-central/rev/beb0aaf0804b
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 34
Verified as fixed in build 35.0a1 (2014-09-08);
Devices: 
Google Nexus 7 (Android 4.4.4);
Samsung Galaxy R (Android 2.3.4).
Status: RESOLVED → VERIFIED
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.