Opened on 03/08/2014 at 08:09:58 AM

Closed on 04/03/2014 at 11:51:15 AM

Last modified on 02/06/2018 at 02:35:27 PM

#4 closed change (fixed)

Add high-DPI versions of Chrome/Safari icons

Reported by: trev Assignee: sebastian
Priority: P4 Milestone: Adblock-Plus-1.8-for-Chrome-Opera-Safari
Module: Platform Keywords:
Cc: sven Blocked By: #3
Blocking: Platform:
Ready: yes Confidential: no
Tester: Verified working: no
Review URL(s):

http://codereview.adblockplus.org/5838171610808320/
http://codereview.adblockplus.org/5700138072801280/
http://codereview.adblockplus.org/5085502482939904/

Description (last modified by trev)

Both Chrome and Safari allow adding high-DPI icons for Retina displays - 38x38 for Chrome and 32x32 for Safari. The agreement here is that we are going with a larger version of the current icon for now, without any additional details.

Attachments (2)

icons_38x38.zip (16.2 KB) - added by sven on 03/11/2014 at 12:50:12 PM.
icons_32x32.zip (15.2 KB) - added by sven on 03/11/2014 at 12:50:26 PM.

Download all attachments as: .zip

Change History (15)

comment:1 Changed on 03/10/2014 at 02:21:57 PM by sven

32x32 and 38x38 icons are attached.

Changed on 03/11/2014 at 12:50:12 PM by sven

Changed on 03/11/2014 at 12:50:26 PM by sven

comment:2 Changed on 03/12/2014 at 02:45:01 PM by trev

  • Description modified (diff)

comment:3 follow-up: Changed on 03/19/2014 at 04:22:12 PM by philll

  • Cc sven added
  • in_progress set to 0
  • Ready unset

@sven: Which of those attachments shall be used? Please specify in the description eg by linking to the desired one.

comment:4 in reply to: ↑ 3 Changed on 03/19/2014 at 04:57:24 PM by philll

Replying to philll:

@sven: Which of those attachments shall be used? Please specify in the description eg by linking to the desired one.

Removed the unnecessary attachments as demanded by sven. Still I recommend to be the description explicit better than implicit and link to the respective attachment where they should be referenced.

comment:5 Changed on 04/02/2014 at 03:18:29 PM by sebastian

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:6 Changed on 04/02/2014 at 04:38:09 PM by trev

  • Status changed from reviewing to reopened

comment:7 Changed on 04/02/2014 at 04:38:23 PM by trev

  • Owner set to sebastian
  • Status changed from reopened to assigned

comment:8 Changed on 04/02/2014 at 04:38:31 PM by trev

  • Status changed from assigned to reviewing

comment:9 Changed on 04/03/2014 at 11:51:15 AM by sebastian

  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:10 Changed on 04/04/2014 at 11:00:12 AM by sebastian

  • Review URL(s) modified (diff)

comment:11 Changed on 04/13/2014 at 06:28:30 PM by trev

  • Component changed from Adblock-Plus-for-Firefox to Platform
  • Milestone set to Adblock-Plus-for-Chrome-Opera-Safari-next

comment:12 Changed on 05/16/2014 at 01:44:13 PM by sebastian

  • Ready set

comment:13 Changed on 02/06/2018 at 02:35:27 PM by abpbot

A commit referencing this issue has landed:
GitLab Issue 4 - Update navigation menus

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from sebastian.
 
Note: See TracTickets for help on using tickets.