Closed Bug 1162253 Opened 8 years ago Closed 8 years ago

Update the Pocket Menu Icon with the correct aspect ratios

Categories

(Firefox :: Pocket, defect, P5)

Product:
Firefox
Component:
Pocket
Version:
38 Branch
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Firefox 40
Iteration:
40.3 - 11 May
Tracking Flags:
Tracking Status
firefox38.0.5 --- fixed
firefox39 --- fixed
firefox40 --- fixed

People

(Reporter: mmaslaney, Assigned: Dolske)

References

Details

Attachments

(3 files, 1 obsolete file)

Patch v.1
1.66 KB, patch
jaws
: review+
Dolske
: approval-mozilla-aurora+
Dolske
: approval-mozilla-release+
Details | Diff | Splinter Review
Screenshot (before)
168.89 KB, image/png
Details
Screenshot (after)
169.10 KB, image/png
Details
Attached file Mark for Menu.zip (obsolete) — 
Please update the Pocket Menu Icon with the attached.
Component: General → Pocket
Blocks: Pocket
Priority: -- → P5
Attached patch Patch v.1 — — Splinter Review 
The provided @2x icon was actually 32x28 instead of 32x32, so I added 2 pixels of padding top&bottom to make it the right size. And I ran it through pngcrush while I was there.
Assignee: nobody → dolske
Attachment #8602331 - Attachment is obsolete: true
Attachment #8603086 - Flags: review?(jaws)
Attached image Screenshot (before) — 

    
    

    
  

      
      
      
      

        Attached image
          
        Screenshot (after)
        — 
      

    


  
Flags: qe-verify-

        Attachment #8603086 -
        Flags: review?(jaws) → review+

    
    

    
  
Comment on attachment 8603086 [details] [diff] [review]
Patch v.1

[Triage Comment]

Required for Pocket / 38.0.5 release.

        Attachment #8603086 -
        Flags: approval-mozilla-release+

        Attachment #8603086 -
        Flags: approval-mozilla-aurora+

    
  
Target Milestone: --- → Firefox 40
Iteration: --- → 40.3 - 11 May
Flags: firefox-backlog+

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