Closed Bug 1048204 Opened 10 years ago Closed 10 years ago

The back button seems like pressed even without pressed

Categories

(Firefox OS Graveyard :: Gaia::SMS, defect)

Product:
Firefox OS Graveyard
Component:
Gaia::SMS
Platform:
ARM
Gonk (Firefox OS)
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(blocking-b2g:2.0+, b2g-v1.3 unaffected, b2g-v1.4 unaffected, b2g-v2.0 verified, b2g-v2.1 verified)

Status:
RESOLVED FIXED
Milestone:
2.1 S2 (15aug)
Project Flags:
blocking-b2g 2.0+
Tracking Flags:
Tracking Status
b2g-v1.3 --- unaffected
b2g-v1.4 --- unaffected
b2g-v2.0 --- verified
b2g-v2.1 --- verified

People

(Reporter: ankit93040, Assigned: azasypkin)

References

Details

(Keywords: regression, Whiteboard: [g+][LibGLA, Dev, B] [p=1])

Attachments

(2 files)

GitHub pull request URL
(deleted), text/x-github-pull-request
steveck
: review+
Details
Verify_video.3gp
(deleted), video/3gpp
Details 
User Agent: Mozilla/5.0 (Windows NT 6.1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/36.0.1985.125 Safari/537.36

Steps to reproduce:

Precondition - 
1. Have a message thread.

Steps -
1. Message app
2. Select a thread enter the thread view.
3. Long click on a text message.
4. click "view message report".
5. click(x) close.
Notice the back button.


Actual results:

back button seems already pressed.


Expected results:

back button shouldn't look like preesed.
Whiteboard: [g+][LibGLA, Dev, B]
Flags: needinfo?(felash)
[Blocking Requested - why for this release]: regression of the visual refresh.

Similar issues were made a blocker in the past (bug 950623).

I checked that it doesn't happen in v1.4. (v1.3 does not have the message report feature).
blocking-b2g: --- → 2.0?
Flags: needinfo?(felash)
Keywords: regression
Patch for bug 968483 is the culprit.
Assignee: nobody → azasypkin
Blocks: 968483
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attached file GitHub pull request URL (deleted) — 
Hey Steve,

Here is the tiny patch that modifies "hover" fix from bug 993388, so that it's compliant with the latest back button styles introduced in bug 968483 :)

Thanks!
Attachment #8467165 - Flags: review?(schung)
Comment on attachment 8467165 [details]
GitHub pull request URL

Thanks for the quick fixing!
Attachment #8467165 - Flags: review?(schung) → review+
(In reply to Steve Chung [:steveck] from comment #4)
> Comment on attachment 8467165 [details]
> GitHub pull request URL
> 
> Thanks for the quick fixing!

Thanks for review!

Master: https://github.com/mozilla-b2g/gaia/commit/aa37e20de9c892f1bc169b40f7c77966b317f027
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
status-b2g-v2.1: affected → fixed
OS: All → Gonk (Firefox OS)
Hardware: All → ARM
Resolution: --- → FIXED
Blocks: sms-sprint-2.1S2
Whiteboard: [g+][LibGLA, Dev, B] → [g+][LibGLA, Dev, B] [p=1]
Target Milestone: --- → 2.1 S2 (15aug)
blocking-b2g: 2.0? → 2.0+
This issue has been verified successfully on Flame 2.0,2.1

See attachment: Verify_video.3gp
Reproducing rate: 0/5
Flame 2.0 versions:
Gaia-Rev        8d1e868864c8a8f1e037685f0656d1da70d08c06
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g32_v2_0/rev/29222e215db8
Build-ID        20141203000201
Version         32.0
Flame 2.1 versions:
Gaia-Rev        dbaf3e31c9ba9c3436e074381744f2971e15c7bf
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/ebce587d2194
Build-ID        20141203001205
Version         34.0
status-b2g-v2.0: fixed → verified
status-b2g-v2.1: fixed → verified
Attached video Verify_video.3gp (deleted) — 

  

  



          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: