Closed
Bug 1147650
Opened 10 years ago
Closed 10 years ago
[Contacts] Quick scrolling via alphabet picker doesn't work properly
Categories
(Firefox OS Graveyard :: Gaia::System, defect)
Tracking
(blocking-b2g:2.5+, b2g-v2.2 unaffected, b2g-master verified)
Tracking | Status | |
---|---|---|
b2g-v2.2 | --- | unaffected |
b2g-master | --- | verified |
People
(Reporter: pcheng, Assigned: kats)
References
Details
(Keywords: regression, Whiteboard: [3.0-Daily-Testing])
Attachments
(2 files)
Description:
Alphabet picker scrolling doesn't work correctly. The touch detection area seems to have shifted a little bit to the left, which makes it unintuitive because it's not directly over the alphabet picker.
STR:
0) Have some contacts in the app
1) Open Contacts
2) Attempt to use the alphabet picker on the right of the screen by sliding finger over the small gray letters
Expected: Quick scrolling works
Actual: Quick scrolling doesn't work over where it should work. If I slide a little bit to the left it works.
See video of issue:
https://www.youtube.com/watch?v=SpRGlDdVrYY
Also attaching a logcat.
Device: Flame 3.0 Master (full flash 319MB KK)
BuildID: 20150325010206
Gaia: aebfbd998041e960cea0468533c0b5041b504850
Gecko: cc0950b7a369
Gonk: b83fc73de7b64594cd74b33e498bf08332b5d87b
Version: 39.0a1 (3.0 Master)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:39.0) Gecko/39.0 Firefox/39.0
Reporter | ||
Comment 1•10 years ago
|
||
This issue doesn't occur on Flame 2.2. Scrolling via alphabet picker works over the expected area.
Device: Flame 2.2
BuildID: 20150325002503
Gaia: aeee2a54caa8ffb875b96264b61d742b70689f22
Gecko: 556aca3e50ac
Gonk: ebad7da532429a6f5efadc00bf6ad8a41288a429
Version: 37.0 (2.2)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0
QA Whiteboard: [QAnalyst-Triage?]
status-b2g-v2.2:
--- → unaffected
status-b2g-master:
--- → affected
Flags: needinfo?(ktucker)
Keywords: regression
Whiteboard: [3.0-Daily-Testing]
Comment 2•10 years ago
|
||
[Blocking Requested - why for this release]:
Nominating this 3.0? since this is a regression from 2.2 and broken functionality.
blocking-b2g: --- → 3.0?
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
Keywords: regressionwindow-wanted
Updated•10 years ago
|
QA Contact: ychung
I have the feeling this has been introduced by bug 1112551. It has landed in 2.2 yesterday. Pi Wei, can you check today's 2.2 to see if this issue has appeared?
Flags: needinfo?(pcheng)
Clearing the regression window flag until we have the confirmation bug 1112551 is or is not the cause. Please add it again once it's not related to bug 1112551.
Keywords: regressionwindow-wanted
Reporter | ||
Comment 5•10 years ago
|
||
This bug doesn't repro on today's 2.2. Seems like bug 1112551 isn't the cause here. Adding window-wanted tag back.
Device: Flame 2.2 (full flash 319MB KK)
BuildID: 20150326002504
Gaia: e59ac067a1d22b7a72cbebc892ec652723f2a557
Gecko: 04b4b9d1faae
Gonk: ebad7da532429a6f5efadc00bf6ad8a41288a429
Version: 37.0 (2.2)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0
Flags: needinfo?(pcheng)
Keywords: regressionwindow-wanted
Comment 6•10 years ago
|
||
Bug 1143518 seems to have caused this issue.
Mozilla-inbound Regression Window
Last Working
Environmental Variables:
Device: Flame 3.0
BuildID: 20150316091433
Gaia: 4868c56c0a3b7a1e51d55b24457e44a7709ea1ae
Gecko: 6a42e09d9a26
Version: 39.0a1 (3.0)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:39.0) Gecko/39.0 Firefox/39.0
First Broken
Environmental Variables:
Device: Flame 3.0
BuildID: 20150316093332
Gaia: 4868c56c0a3b7a1e51d55b24457e44a7709ea1ae
Gecko: 3d8fb4f958b5
Version: 39.0a1 (3.0)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:39.0) Gecko/39.0 Firefox/39.0
Last Working gaia / First Broken gecko - Issue DOES occur
Gaia: 4868c56c0a3b7a1e51d55b24457e44a7709ea1ae
Gecko: 3d8fb4f958b5
First Broken gaia / Last Working gecko - Issue does NOT occur
Gaia: 4868c56c0a3b7a1e51d55b24457e44a7709ea1ae
Gecko: 6a42e09d9a26
Gecko Pushlog: http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=6a42e09d9a26&tochange=3d8fb4f958b5
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage?]
Flags: needinfo?(ktucker)
Keywords: regressionwindow-wanted
Comment 7•10 years ago
|
||
Kartikaya, can you take a look at this please? Looks like this was caused by the landing for bug 1143518.
Blocks: 1143518
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker) → needinfo?(bugmail.mozilla)
Assignee | ||
Comment 9•10 years ago
|
||
The touch forwarder is eating the events. Prior to bug 1143518 we were bypassing the touch forwarder because of the event capture code path (which was technically wrong because listeners should receive all the events they're listening for). With that gone this bug got exposed. Patch coming once github decides to wake up and accept my push.
Assignee: nobody → bugmail.mozilla
Component: Gaia::Contacts → Gaia::System
Flags: needinfo?(bugmail.mozilla)
Comment 10•10 years ago
|
||
Assignee | ||
Updated•10 years ago
|
Attachment #8584535 -
Flags: review?(etienne)
Comment 11•10 years ago
|
||
Comment on attachment 8584535 [details]
[gaia] staktrace:forwarder > mozilla-b2g:master
cool :)
Attachment #8584535 -
Flags: review?(etienne) → review+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Updated•10 years ago
|
Keywords: checkin-needed
Comment 13•10 years ago
|
||
Pull request has landed in master: https://github.com/mozilla-b2g/gaia/commit/a5ba1aa61fee10d9b9e69998af929015bb77e06c
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•10 years ago
|
Flags: needinfo?(kgrandon)
Comment 14•10 years ago
|
||
(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #12)
> Autolander, y u no land?
Sorry, it can be a bit slow to update. I'm looking at ways to improve the usability. It's easy enough to comment in the bug, but I'm not sure if that would cause too much churn.
Assignee | ||
Comment 15•10 years ago
|
||
I would have been happy with a comment when it removed the checkin-needed keyword.
Updated•10 years ago
|
Target Milestone: --- → 2.2 S9 (3apr)
Comment 16•10 years ago
|
||
This issue is verified fixed on the latest Nightly Flame 3.0 build.
Actual Results: The user can scroll along the Alphabet picker as expected.
Environmental Variables:
Device: Flame 3.0
BuildID: 20150506010204
Gaia: 3e6fd1e0a478af2c95d09ce95c2c6de2de2fec14
Gecko: ba44099cbd07
Gonk: a9f3f8fb8b0844724de32426b7bcc4e6dc4fa2ed
Version: 40.0a1 (3.0)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:40.0) Gecko/40.0 Firefox/40.0
Status: RESOLVED → VERIFIED
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage?]
Flags: needinfo?(ktucker)
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
Updated•9 years ago
|
status-b2g-v2.5:
--- → verified
Updated•9 years ago
|
status-b2g-v2.5:
verified → ---
You need to log in
before you can comment on or make changes to this bug.
Description
•