Closed
Bug 1385988
Opened 7 years ago
Closed 7 years ago
Highlights item separators disappear from the top when scrolling down
Categories
(Firefox for Android Graveyard :: General, defect, P1)
Tracking
(fennec+, firefox55 unaffected, firefox56 disabled, firefox57 verified)
VERIFIED
FIXED
Firefox 57
Tracking | Status | |
---|---|---|
fennec | + | --- |
firefox55 | --- | unaffected |
firefox56 | --- | disabled |
firefox57 | --- | verified |
People
(Reporter: mcomella, Assigned: sebastian)
References
Details
(Whiteboard: [mobileAS])
Attachments
(2 files)
STR: - With enough highlights that you need to scroll... - Scroll down to see more highlights Expected: item separators remain Actual: item separators disappear I'm guessing our math is off when we remove the item decoration separators in the recyclerView for the Highlights title & friends.
Reporter | ||
Updated•7 years ago
|
Priority: P1 → --
Updated•7 years ago
|
No longer blocks: as-android-blockers
Priority: -- → P3
Updated•7 years ago
|
tracking-fennec: ? → +
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → s.kaspari
Status: NEW → ASSIGNED
Comment hidden (mozreview-request) |
Reporter | ||
Comment 3•7 years ago
|
||
mozreview-review |
Comment on attachment 8905985 [details] Bug 1385988 - HighlightsDividerItemDecoration: Use adapter position and not current position in parent view. https://reviewboard.mozilla.org/r/177742/#review182848
Attachment #8905985 -
Flags: review?(michael.l.comella) → review+
Reporter | ||
Updated•7 years ago
|
Iteration: --- → 1.29
Priority: P3 → P1
Pushed by michael.l.comella@gmail.com: https://hg.mozilla.org/integration/autoland/rev/ae513db6d626 HighlightsDividerItemDecoration: Use adapter position and not current position in parent view. r=mcomella
Comment 6•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/ae513db6d626
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox57:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
Updated•7 years ago
|
status-firefox55:
--- → unaffected
status-firefox56:
--- → disabled
Comment 7•7 years ago
|
||
Verified this using the steps in Comment 0 on a Nexus 5 (Android 6.0.1).
Status: RESOLVED → VERIFIED
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
•