[Flame][V1.4][Dialer]There is no prompts when you tap on "Call" without any number in the dialer field.

RESOLVED WONTFIX

Status

defect
RESOLVED WONTFIX
5 years ago
Last year

People

(Reporter: panda67231, Unassigned)

Tracking

unspecified
ARM
Gonk (Firefox OS)
Dependency tree / graph

Firefox Tracking Flags

(b2g-v2.2 affected)

Details

(Whiteboard: [lang=js][lang=css])

Attachments

(5 attachments, 3 obsolete attachments)

Posted file attachment.rar
[1.Description]: 
When there is no MO call history, tap on "Call" button, user can't be informed no dialing action will be executed.

Attach the video: Video_Redial.mp4
Attach the logs: bugreport_Redial.txt & logcat_Redial.txt

Happened time: 5:00PM

[2.Testing Steps]: 
Preconditions: a. There is no MO call history; 2. Dialer field is empty - no number has been typed in the field  
1. Launch Dialer app 
2.. Tap on "Call" with out any number in the dialer field 

[3.Expected Result]: 
2.User should be informed no dialing action will be executed  

[4.Actual Result]: 
2.There is no prompts when you tap on "Call" without any number in the dialer field.

[5.Reproduction build]: 
Gaia        7709936aeb21859d1607dbd038489493803bb085
Gecko     https://hg.mozilla.org/releases/mozilla-b2g30_v1_4/rev/5bf038fae0f1
BuildID    20140522160202
Version    30.0

[6.Reproduction Frequency]: Always Recurrence,5/5
This issue still occurs on master.

We could fix it by just greying out the call button if there is both: 1. no call history, and 2. no digits entered. Or we could display an error message when tapping the call button. Carrie, what do you think?
Mentor: drs.bugzilla
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: needinfo?(cawang)
Whiteboard: bamboo → [good first bug][lang=js][lang=css]
Hi, please grey out the button rather than displaying an error message. It will be too much and looks like something really bad just happened. Thanks!
Flags: needinfo?(cawang)
Agreed that to grey out is a better idea. 

After checking the codes, looks like the ID of the button is "keypad-callbar-call-action". But, don't know how to do the control here.
Ok, so I have managed to disable the button, but I am not sure how to grey it out. Is there any 'disabled caller' icon available ?
Comment on attachment 8542546 [details] [review]
Disable the call button if the dialer field is empty

Simple and clean. However, this needs some tests added as I mentioned in the PR. If you need help with this, let me know.

Please also post a screenshot of the disabled button and flag Carol Huang (:Carol) for ui-review.

Thanks for your participation!
Flags: needinfo?(rohan1395)
Attachment #8542546 - Flags: review?(drs.bugzilla) → review-
Posted image Disabled call button (obsolete) —
Yes Doug, I would like your help with testing. I am available on #gaia right now
Flags: needinfo?(rohan1395)
Attachment #8542819 - Flags: ui-review?(chuang)
Comment on attachment 8542819 [details]
Disabled call button

Hi,
The call disable state should be the same as the "add to contact" button(disable state) on the left.Thanks!!
Attachment #8542819 - Flags: ui-review?(chuang) → ui-review-
Posted image Disabled call button (obsolete) —
Attachment #8542819 - Attachment is obsolete: true
Attachment #8545864 - Flags: ui-review?(chuang)
Posted file images_0113.zip (obsolete) —
Hey Rohan,
Could you help replace the call icon for the disabled state!Thank you!!
Sorry, I thought it was a SVG file :P
Flags: needinfo?(rohan1395)
Comment on attachment 8545864 [details]
Disabled call button

please replace the new 'Call' images, thanks!
Attachment #8545864 - Flags: ui-review?(chuang) → ui-review-
Rohan, do you still need help with this?
Posted image Error.png
Carol, as you can see from the attachment, the disabled call button(grey) is smaller than the active button(white). I am not sure if this is right.
Flags: needinfo?(rohan1395) → needinfo?(chuang)
Posted file images.zip
Hi Rohan,
Could you help me replace the call icons?
I've updated the 2 states of the images. (normal/disable)

Let me know if there's any other question.
Thanks so much!!
Attachment #8547922 - Attachment is obsolete: true
Flags: needinfo?(chuang) → needinfo?(rohan1395)
Hey Carol, I have replaced the call icons with the one you attached. PLease review.
Attachment #8545864 - Attachment is obsolete: true
Flags: needinfo?(rohan1395)
Attachment #8550253 - Flags: ui-review?(chuang)
Comment on attachment 8542546 [details] [review]
Disable the call button if the dialer field is empty

Hello Doug, I have made the changes as you requested. Please review.
Attachment #8542546 - Flags: review- → review?(drs.bugzilla)
Comment on attachment 8542546 [details] [review]
Disable the call button if the dialer field is empty

Thanks for sticking with this! Unfortunately, we need some more tests to get r+. If you have any questions, feel free to needinfo me or ping me on IRC. My nick is "drs".
Attachment #8542546 - Flags: review?(drs.bugzilla) → review-
Comment on attachment 8550253 [details]
Disabled and enabled call button

Thank you, Rohan!
Attachment #8550253 - Flags: ui-review?(chuang) → ui-review+
I am solving this
Comment on attachment 8542546 [details] [review]
Disable the call button if the dialer field is empty

I have added some unit tests
Attachment #8542546 - Flags: review- → review?(drs.bugzilla)
Comment on attachment 8542546 [details] [review]
Disable the call button if the dialer field is empty

Rohan, please add the 'checkin-needed' keyword to this bug when you're ready to check this in.
Attachment #8542546 - Flags: review?(drs.bugzilla) → review+
Assignee: nobody → rohan1395
Keywords: checkin-needed
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Backed out for causing the exact same Gaia unit test failures that were visible in the Gaia Try run on the Pull Request. Please verify your test runs before landing in the future.

Master: https://github.com/mozilla-b2g/gaia/commit/eff3321ab4e65da3f906688ebb55ddf1e93d9452
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Sorry, that was my fault for a) not checking that the tests all pass and b) not explaining that we need to look at them and fix them if they're failing.

Rohan, you can check your test results on the PR. For example, it says "Rohan Rawat (rohan013) started tests. Results"

Sometimes you will get an orange or red when there's actually nothing wrong, or your patch isn't at fault. If this happens, ask me and we'll go over it together to be sure. Otherwise, if one of the tests that is affected by your patch is failing, you need to fix it before landing.
Flags: needinfo?(rohan1395)
Assignee: rohan1395 → nobody
Mentor: bugzilla
Flags: needinfo?(rohan1395)
Whiteboard: [good first bug][lang=js][lang=css] → [lang=js][lang=css]
Firefox OS is not being worked on
Status: REOPENED → RESOLVED
Closed: 4 years agoLast year
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.