Closed Bug 1269027 Opened 8 years ago Closed 8 years ago

Add empty state for no synced devices

Categories

(Firefox for Android Graveyard :: General, defect)

ARM
Android
defect
Not set
normal

Tracking

(firefox48+ verified, firefox49 verified)

RESOLVED FIXED
Firefox 49
Tracking Status
firefox48 + verified
firefox49 --- verified

People

(Reporter: liuche, Assigned: liuche)

References

Details

(Keywords: regression)

Attachments

(4 files)

Add a hacky version of "Sync Setup" to empty history-sync panel.

Attaching a screenshot for a WIP patch for sync empty state. This has some edge cases in landscape mode, or on small phones.
This is built on top of the commits from bug 1261527 but reviewboard doesn't allow pushing from commit series with more than one bug number.
Assignee: nobody → liuche
Attachment #8747387 - Flags: review?(s.kaspari)
Barbara, if we want to get the empty state work into 48, we can uplift the patches from this bug and the bug it's built upon (bug 1261527).
Flags: needinfo?(bbermes)
This is a hacky patch to get the empty states to appear underneath the RecyclerView that uses weights because the RecyclerView library in 23.0 doesn't handle wrap_content.

When using ViewStubs, I was running into a lot of trouble with the inflated views disappearing during device rotation so I just dropped them and replaced them with normal views. I could also turn them into <merge> views, if that's better?

I'm still not sure what was going on with the view stubs, but I did some debugging inside of onLayout and the RecyclerView just expanded to fill all the space. Perhaps the layout dropped the views that replaced the viewstubs because they weren't kept track of in the main layout for some reason?
Sounds good Chenxia and thanks for finding a way to include this.
Flags: needinfo?(bbermes)
Attachment #8747387 - Flags: review?(s.kaspari) → review+
(In reply to Chenxia Liu [:liuche] from comment #1)
> Created attachment 8747387 [details] [diff] [review]
> Patch: Empty state
> 
> This is built on top of the commits from bug 1261527 but reviewboard doesn't
> allow pushing from commit series with more than one bug number.

I think you can do this with "hg push -c <rev> review".
https://hg.mozilla.org/mozilla-central/rev/f410732bf7cf
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
Comment on attachment 8747387 [details] [diff] [review]
Patch: Empty state

Approval Request Comment
[Feature/regressing bug #]: Sync empty state was removed in bug 1220928.
[User impact if declined]: No entry point into sync setup
[Describe test coverage new/current, TreeHerder]: Local, nightly
[Risks and why]: low, reuses sync empty state from previous version
[String/UUID change made/needed]: none

Depends on bug 1261527.
Attachment #8747387 - Flags: approval-mozilla-aurora?
Comment on attachment 8747387 [details] [diff] [review]
Patch: Empty state

Fix for regression from 48.
Attachment #8747387 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Verified as fixed using:
Device: Moto X (Android 4.4)
Build: Firefox for Android 49.0a1 (2016-05-04)
This patch doesn't apply to aurora.

$ hg graft -er f410732bf7cf
grafting 341185:f410732bf7cf "Bug 1269027 - Add empty state for no synced devices. r=sebastian"
merging mobile/android/base/java/org/mozilla/gecko/home/CombinedHistoryAdapter.java
merging mobile/android/base/java/org/mozilla/gecko/home/CombinedHistoryPanel.java
merging mobile/android/base/resources/layout/home_combined_history_panel.xml
warning: conflicts while merging mobile/android/base/java/org/mozilla/gecko/home/CombinedHistoryAdapter.java! (edit, then use 'hg resolve --mark')
warning: conflicts while merging mobile/android/base/java/org/mozilla/gecko/home/CombinedHistoryPanel.java! (edit, then use 'hg resolve --mark')
warning: conflicts while merging mobile/android/base/resources/layout/home_combined_history_panel.xml! (edit, then use 'hg resolve --mark')
abort: unresolved conflicts, can't continue
(use hg resolve and hg graft --continue)
Flags: needinfo?(liuche)
Wes, this bug depends on bug 1261527, so that needs to be uplifted first. I requested approval, so we should wait on that. Is there a releng way I should message that? I added it to the aurora request comment, and also added in the bug dependency, but let me know if there is better way I can flag this for releng?
Flags: needinfo?(liuche) → needinfo?(wkocher)
Verified as fixed using:
Device: Motorola Nexus 6 (Android 6.0)
Build: Firefox for Android 48.0a2 (2016-05-23)
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.