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)
Tracking
(firefox48+ verified, firefox49 verified)
RESOLVED
FIXED
Firefox 49
People
(Reporter: liuche, Assigned: liuche)
References
Details
(Keywords: regression)
Attachments
(4 files)
77.10 KB,
image/png
|
Details | |
15.76 KB,
patch
|
sebastian
:
review+
lizzard
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
60.09 KB,
image/png
|
Details | |
105.68 KB,
image/png
|
Details |
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.
Assignee | ||
Comment 1•8 years ago
|
||
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)
Assignee | ||
Comment 2•8 years ago
|
||
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)
Assignee | ||
Comment 3•8 years ago
|
||
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?
Comment 4•8 years ago
|
||
Sounds good Chenxia and thanks for finding a way to include this.
Flags: needinfo?(bbermes)
Updated•8 years ago
|
Attachment #8747387 -
Flags: review?(s.kaspari) → review+
Comment 5•8 years ago
|
||
(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".
Comment 7•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/f410732bf7cf
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
Assignee | ||
Comment 8•8 years ago
|
||
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 9•8 years ago
|
||
Comment on attachment 8747387 [details] [diff] [review] Patch: Empty state Fix for regression from 48.
Attachment #8747387 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Updated•8 years ago
|
Comment 10•8 years ago
|
||
Verified as fixed using: Device: Moto X (Android 4.4) Build: Firefox for Android 49.0a1 (2016-05-04)
Updated•8 years ago
|
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)
Assignee | ||
Comment 12•8 years ago
|
||
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)
Comment 13•8 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/b14d8e23323e
Flags: needinfo?(wkocher)
Comment 14•8 years ago
|
||
Verified as fixed using: Device: Motorola Nexus 6 (Android 6.0) Build: Firefox for Android 48.0a2 (2016-05-23)
Updated•8 years ago
|
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•