1145228 - Allow collapsing and expanding Synced (Remote) Tabs clients

Status: RESOLVED DUPLICATE of bug 1345279

(Firefox for iOS :: Home screen, defect)

Description

[Nick Alexander :nalexander [he/him]]

Duplicate of the functionality on Android.  See tecgirl's beautiful screenshot:

http://cl.ly/aHie

There are few parts to this.  We need to:

* add and persist state for whether a client is collapsed;
* draw the appropriate arrows in the header views;
* add the interaction and animations for expanding and collapsing.

Mentor bug!

Comment 1

[Jared S [:BigxMac]]

I've been working on this. Can I officially take the bug?

Comment 2

[Aaron Train [:aaronmt]]

Absolutely!

Comment 3

[Robin Andersen [:tecgirl]]

:BigxMac Where are we at with this? I was just filing a fresh bug for it.

Slightly updated mock: https://invis.io/EC2U2YZB4

Comment 4

[Brian Nicholson (:bnicholson)]

Unassigning due to inactivity. BigxMac, feel free to take again if you're working on it.

Comment 5

[Robin Andersen [:tecgirl]]

Attachment: syncCaret.zip

Comment 6

[Bryan Munar (:bkmunar)]

in da process of a fix!

Comment 7

[Bryan Munar (:bkmunar)]

Attachment: PR - make bnicholz fix the rest // my parting gift

TODOs:

1. make the caret animation and placement UI-approved
2. make sure the offset doesn't screw up where the user is on the tabs panel (aka when a row is collapsed, the tableView doesn't scroll down 16 rows or whatever).
3. It crashes when you press the caret after the initial pull to refresh; after that, it doesn't crash anymore.

Comment 8

[Robin Andersen [:tecgirl]]

Comment on attachment 8652492 [details] [review]
PR - make bnicholz fix the rest // my parting gift

Two nits:

1) The caret is vertically-bottom aligned. Please vertically-center align it.
2) Add a bottom border to the headers so when they are collapsed they don't bleed together. The border we use on the History panel headers (Today, Yesterday, etc.) for time would be perfect.

Comment 9

[Brian Nicholson (:bnicholson)]

Comment on attachment 8652492 [details] [review]
PR - make bnicholz fix the rest // my parting gift

Removing review flag since there's a good amount of work left outlined in comment 7 and comment 8.

Comment 10

[Stephan Leroux [:sleroux]]

PR became dormant: https://github.com/mozilla/firefox-ios/pull/943

Closed for now but can be reopened if we plan on coming back to this feature.