Closed
Bug 1120882
Opened 10 years ago
Closed 10 years ago
(App-grouping) Dragging the last expanded group with one row of icons causes temporary incorrect offset
Categories
(Firefox OS Graveyard :: Gaia::Homescreen, defect)
Tracking
(b2g-v2.2 verified, b2g-master verified)
VERIFIED
FIXED
2.2 S8 (20mar)
People
(Reporter: cwiiis, Assigned: cwiiis)
References
Details
(Keywords: polish, Whiteboard: [systemsfe])
Attachments
(1 file)
(deleted),
text/x-github-pull-request
|
kgrandon
:
review+
bajaj
:
approval-gaia-v2.2+
|
Details |
If you have an expanded group at the end of a scrollable grid of icons with only one row of icons, and you scroll to the bottom, then long-press its drag handle to collapse it, the grid will be forced to scroll (as it has reduced in size) and the collapsed group will temporarily show with an incorrect offset. As soon as you move your finger again, it will be correct.
The scroll position changes are also quite janky (as the main thread isn't updated as frequently as the compositor, and even if it was, would be unlikely to keep up) - this is a separate bug though, which I will file. The fix may end up fixing both, however.
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → chrislord.net
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Comment 1•10 years ago
|
||
Assignee | ||
Comment 2•10 years ago
|
||
Comment on attachment 8573365 [details]
[gaia] Cwiiis:bug1120882-dont-resize-during-drag > mozilla-b2g:master
This fixes the problem by just not resizing the grid during a drag. Because the grid never gets larger during a drag and the grid is always re-rendered after a drag, we can just skip sending the resize signal during drags and rely on the render-on-finish to re-send it.
Attachment #8573365 -
Flags: review?(kgrandon)
Comment 3•10 years ago
|
||
Comment on attachment 8573365 [details]
[gaia] Cwiiis:bug1120882-dont-resize-during-drag > mozilla-b2g:master
This seems like there's a hard dependency on bug 1120884 which still has some questions there, so clearing review for now. Can you either roll this into that bug, or flag me after bug 1120884 has landed?
Attachment #8573365 -
Flags: review?(kgrandon)
Assignee | ||
Updated•10 years ago
|
Attachment #8573365 -
Flags: review?(kgrandon)
Comment 4•10 years ago
|
||
Comment on attachment 8573365 [details]
[gaia] Cwiiis:bug1120882-dont-resize-during-drag > mozilla-b2g:master
Nice and smooth, thanks!
Attachment #8573365 -
Flags: review?(kgrandon) → review+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Updated•10 years ago
|
Keywords: checkin-needed
Comment 5•10 years ago
|
||
http://docs.taskcluster.net/tools/task-graph-inspector/#ONlamdpnTyWMdg2MxZX3SA
The pull request failed to pass integration tests. It could not be landed, please try again.
Assignee | ||
Comment 6•10 years ago
|
||
Sorry autolander (tests are green, one unrelated intermittent that passed on a retrigger) - merged manually: https://github.com/mozilla-b2g/gaia/commit/3325c49b8ca9b238399acf045efbbb7993269d66
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 7•10 years ago
|
||
(In reply to Chris Lord [:cwiiis] from comment #6)
> Sorry autolander (tests are green, one unrelated intermittent that passed on
> a retrigger) - merged manually:
> https://github.com/mozilla-b2g/gaia/commit/
> 3325c49b8ca9b238399acf045efbbb7993269d66
Autolander/taskcluster must be drunk. Logs are not too helpful here, I've filed bug 1142130 to track this down.
Updated•10 years ago
|
Target Milestone: --- → 2.2 S7 (6mar)
Assignee | ||
Comment 8•10 years ago
|
||
Comment on attachment 8573365 [details]
[gaia] Cwiiis:bug1120882-dont-resize-during-drag > mozilla-b2g:master
[Approval Request Comment]
[Bug caused by] (feature/regressing bug #):
[User impact] if declined: Confusing/bad-looking behaviour when dragging the last group on the homescreen (or any group that causes the grid to shrink when scrolled to the bottom).
[Testing completed]: Manual testing and been on master for a while.
[Risk to taking this patch] (and alternatives if risky): Very low risk.
[String changes made]: None.
Attachment #8573365 -
Flags: approval-gaia-v2.2?
Updated•10 years ago
|
Attachment #8573365 -
Flags: approval-gaia-v2.2? → approval-gaia-v2.2+
Comment 10•10 years ago
|
||
status-b2g-v2.2:
--- → fixed
status-b2g-master:
--- → fixed
Target Milestone: 2.2 S7 (6mar) → 2.2 S8 (20mar)
Comment 11•10 years ago
|
||
Verify this bug with below builds
*2.2
Build ID 20150323162503
Gaia Revision e54c4ed1cc188f70ddf1156534d364005dc45490
Gaia Date 2015-03-23 19:09:26
Gecko Revision https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/7ba1778d237b
Gecko Version 37.0
Device Name flame
Firmware(Release) 4.4.2
Firmware(Incremental) eng.cltbld.20150323.200543
Firmware Date Mon Mar 23 20:05:54 EDT 2015
Bootloader L1TC000118D0
*master
Serial: f2cfccfb (State: device)
Build ID 20150323160205
Gaia Revision 8eac260ee81a8aca05770d18c5736536d44ee7a7
Gaia Date 2015-03-23 02:41:53
Gecko Revision https://hg.mozilla.org/mozilla-central/rev/bc85c479668a
Gecko Version 39.0a1
Device Name flame
Firmware(Release) 4.4.2
Firmware(Incremental) eng.cltbld.20150323.193421
Firmware Date Mon Mar 23 19:34:34 EDT 2015
Bootloader L1TC100118D0
You need to log in
before you can comment on or make changes to this bug.
Description
•