1085256 - [Midori 2.0][Dialer]Volume of press number disappear when change volume quickly

Status: RESOLVED DUPLICATE of bug 834530

(Firefox OS Graveyard :: Gaia::Dialer, defect, P2)

Description

[sync-1]

FFOS2.0 baseline BuildID: 20140916000205
 
 DEFECT DESCRIPTION:
  Volume of press number  disappear when change volume quickly
  REPRODUCING PROCEDURES:
  1.enter to call
  2.press volume key then change volume from minimum to maximuun
  3.volume of press number from mute to a voice and no change->KO1
  4.change volume quickly when press number
  5.volume of press number disappear-->KO2  
 
 
  EXPECTED BEHAVIOUR:
  KO1/KO2:volume of press key should change with changing volume
 
  ASSOCIATE SPECIFICATION:
 
  TEST PLAN REFERENCE:
 
  TOOLS AND PLATFORMS USED:
 
  USER IMPACT:
  high
  REPRODUCING RATE:
  5/5
  For FT PR, Please list reference mobile's behavior:

Comment 1

[sync-1]

Attachment: jrdlog

Comment 2

[sync-1]

Attachment: vedio

Comment 3

[Johan Lorenzo [:jlorenzo]]

Should the keypad volume change depending on the overall volume?

Comment 4

[Carrie Wang [New account: [e-mail]]]

Yes, currently we have three types of volume adjustment, ringtone/system, music and alarm. The dialerpad volume depends on the ringtone/system. Thanks!

Comment 5

[Johan Lorenzo [:jlorenzo]]

QA wanted for a branch check.

Comment 6

[Lingling Zhao]

Dears,

Any updates on this problem?

Comment 7

[Gabriele Svelto [:gsvelto]]

Just checked the video and this is a dup of bug 834530. It will soon be fixed on master.

Comment 8

[Lingling Zhao]

OK, so please remember to merge the patch on v2.0.

Thanks.

Comment 9

[Lingling Zhao]

Dear Gabriele,

There are so many comments in bug 834530 that I'm not very clear for the patches. 
Does the patch mentioned in https://bugzilla.mozilla.org/show_bug.cgi?id=834530#c101 is enough for this issue? Is there any new issue caused by that patch? I also see there are conflict in the patch in https://github.com/mozilla-b2g/gaia/pull/25954.

Comment 10

[Gabriele Svelto [:gsvelto]]

The patch has already landed on master so if you're using that you should already have the fix. If you're on v2.1 please use the patch here:

https://github.com/mozilla-b2g/gaia/pull/25900

If you're on 2.0 use this one instead:

https://github.com/mozilla-b2g/gaia/pull/25954

The mentions of conflicts in the commit message means only that when I tried to cherry-pick the patch git has encountered some conflicts (which are listed there). I have solved all of them manually thus you will be able to apply the patch cleanly. I hope this helps.