Closed
Bug 1094709
Opened 10 years ago
Closed 10 years ago
[Flame][Contacts]Tap "x" in left of Name Input Box when edit a contact, it will not delete the name.
Categories
(Firefox OS Graveyard :: Gaia::Contacts, defect)
Tracking
(b2g-v2.1 affected, b2g-v2.2 verified)
VERIFIED
FIXED
2.2 S1 (5dec)
People
(Reporter: zikui.yang, Assigned: jmcf)
References
Details
(Whiteboard: [p=2])
Attachments
(2 files, 1 obsolete file)
[1.Description]:
Tap "x" in left of Name Input Box when editing a contact, it will not delete the name.
video:VIDEO0340_Compress.MP4
log:logcat.txt
happen time:3:43
[2.Testing Steps]:
1.Launch contact
2.Tap Add icon to enter contact edit view
3.Input name or last name to input box
4.Tap "x" in name input box
[3.Expected Result]:
4.it will delete the name.
[4.Actual Result]:
4.it will not delete the name.
[5.Reproduction build]:
Gaia-Rev 154da5e17029a51002d5d9b7df39563d509edde6
Gecko-Rev https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/3b0c3580a58d
Build-ID 20141105001204
Version 34.0
[6.Reproduction Frequency]:
occasionally Recurrence,5/10
Assignee | ||
Updated•10 years ago
|
Target Milestone: --- → 2.1 S9 (21Nov)
Updated•10 years ago
|
Assignee: nobody → ferjmoreno
Whiteboard: [p=2]
Comment 3•10 years ago
|
||
Thanks for the video Elie.
It seems that this is not happening on master anymore, I can only reproduce it on 2.1. This makes me wonder if this is worth fixing given the stage we are wrt accepting patches for 2.1. Francisco, what do you think?
In any case, this looks like a building blocks issue. Any idea of what might have change between 2.1 and master?
Flags: needinfo?(rnowmrch)
Flags: needinfo?(francisco)
As discussed with Fernando previously, clear button has no functionality in BB, but the button is showed when the input as focus, that means when you tap on the icon the button hides as the input blurs :)
Not sure what could have changed between 2.0 to 2.1 and 2.1 to current master.
Would be cool to implement something similar to -webkit-search-cancel-button to have this behavior supported in gecko.
Another way to improve this would be creating a Web Component.
Flags: needinfo?(rnowmrch)
Assignee | ||
Comment 5•10 years ago
|
||
(In reply to Fernando Jiménez Moreno [:ferjm] from comment #3)
> Thanks for the video Elie.
>
> It seems that this is not happening on master anymore, I can only reproduce
> it on 2.1. This makes me wonder if this is worth fixing given the stage we
> are wrt accepting patches for 2.1. Francisco, what do you think?
>
I got some approvals of simple staff for v2.1. Maybe we can try it. The bug it is a bit ugly
> In any case, this looks like a building blocks issue. Any idea of what might
> have change between 2.1 and master?
Comment 6•10 years ago
|
||
Agree with Jose, let's try to fix it as it's pretty ugly and can become a blocker.
Flags: needinfo?(francisco)
Comment 7•10 years ago
|
||
I tried but I am basically CSS blind :( So I am unassigning myself from this issue.
Assignee: ferjmoreno → nobody
Assignee | ||
Comment 9•10 years ago
|
||
I've looking into this issue and I've found that the issue is that the 'x' button is too close to the right, which basically means that it is very easily that the user can press outside the button and as a result the input field would lose the focus and the effect would be equivalent as if the 'x' button didn't work, but it is actually working.
My proposal here would be the following:
Hide the Gecko ScrollBar in the 'Contact Form Screen' as we do in the Contact List screen. Then, try to increase the affordance of the 'x' to the right as much as possible, so that if the user clicks on the right the button will respond.
I'm now working on this approach
Assignee | ||
Updated•10 years ago
|
Summary: [Flame][v2.1][Contacts]Tap "x" in left of Name Input Box when edit a contact, it will not delete the name. → [Flame][Contacts]Tap "x" in left of Name Input Box when edit a contact, it will not delete the name.
Assignee | ||
Comment 10•10 years ago
|
||
The proposed patch increase the affordance of the reset button in those input fields that are on rightmost part of the view (name, last name, org).
Attachment #8523850 -
Flags: review?(crdlc)
Comment 11•10 years ago
|
||
Comment on attachment 8523850 [details]
26192.html
LGTM, thanks
Attachment #8523850 -
Flags: review?(crdlc) → review+
Assignee | ||
Comment 12•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 13•10 years ago
|
||
Once QA verifies in master, we will ask for an uplift to v2.1 .
Keywords: verifyme
Comment 14•10 years ago
|
||
Reverted for test_add_photo_to_contact.py failures.
https://treeherder.mozilla.org/ui/logviewer.html#?job_id=853190&repo=b2g-inbound
Master: https://github.com/mozilla-b2g/gaia/commit/ae3a84acaab80a5b35d5542d63e68462273c8a1b
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 15•10 years ago
|
||
I'm not able to reproduce the test failure locally and I have my doubts that my patch is responsible for the failure, Zac, any clue?
Flags: needinfo?(zcampbell)
Comment 16•10 years ago
|
||
(In reply to Jose Manuel Cantera from comment #15)
> I'm not able to reproduce the test failure locally and I have my doubts that
> my patch is responsible for the failure, Zac, any clue?
As this failed identically on both Linux and Mac I am 100% sure that your patch caused it.
You need to update the wait in the EditContact __init__ block. It is expecting the Edit Contact screen to slide in (hence waiting for y == 0) . Has your patch changed this behaviour? The test is probably tapping on the contact picture before the EditContact has finished animating.
Flags: needinfo?(zcampbell)
Assignee | ||
Comment 17•10 years ago
|
||
I need to investigate more in order to solve the test issue. It seems we need to enlarge the ckick area around the contact photo edition button ...
Updated•10 years ago
|
Target Milestone: 2.1 S9 (21Nov) → 2.2 S1 (5dec)
Assignee | ||
Comment 18•10 years ago
|
||
the same patch but with a pointer-events to enable correct event delivery to the contact photo region
Attachment #8527878 -
Flags: review?(crdlc)
Assignee | ||
Updated•10 years ago
|
Attachment #8523850 -
Attachment is obsolete: true
Comment 19•10 years ago
|
||
Comment on attachment 8527878 [details]
26326.html
LGTM, thanks a lot
Attachment #8527878 -
Flags: review?(crdlc) → review+
Assignee | ||
Comment 20•10 years ago
|
||
Status: REOPENED → RESOLVED
Closed: 10 years ago → 10 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
status-b2g-v2.1:
--- → affected
Comment 21•10 years ago
|
||
Verified the issue is fixed on 2.2 Flame
Tapping on the "X" button removes the first or last name
"Flame 2.2
Environmental Variables:
Device: Flame 2.2 (319mb)(Kitkat Base)(Full Flash)
BuildID: 20141201040205
Gaia: 39214fb22c203e8849aaa1c27b773eeb73212921
Gecko: 08be3008650f
Gonk: 48835395daa6a49b281db62c50805bd6ca24077e
Version: 37.0a1 (Unknown)
Firmware: V188-1
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0
====================================================================
Leaving "verifyme" for 2.1
Status: RESOLVED → VERIFIED
QA Whiteboard: [QAnalyst-Triage?]
status-b2g-v2.2:
--- → verified
Flags: needinfo?(ktucker)
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
You need to log in
before you can comment on or make changes to this bug.
Description
•