Opened 5 years ago

Closed 5 years ago

#1535 closed change (fixed)

Replace ABP icon with new one (v.1.1) - for Chrome/Opera/Safari

Reported by: sven Assignee: sebastian
Priority: P3 Milestone: Adblock-Plus-1.8.11-for-Chrome-Opera-Safari
Module: User-Interface Keywords:
Cc: trev, sebastian Blocked By:
Blocking: #1533 Platform: Chrome
Ready: no Confidential: no
Tester: Verified working: no
Review URL(s):

http://codereview.adblockplus.org/6200136564736000
http://codereview.adblockplus.org/5071126564700160

Description (last modified by sven)

Background

We've created a new icon that was announced and discussed in our ​blog and in our ​forum. After some improvements the 1.1 and 1.2 versions are ready to implement now.

What to change

In general: Replace the current ABP icons with the new ones.
In detail:

filenameplatformnewfile
icons/abp-16.pngchromium/safariactive_16_v_1_1.png
icons/abp-32.pngchromium/safariactive_32_v_1_1.png
icons/abp-128.pngchromium/safari128px-active_v1_2.png
chrome/icons/abp-19.pngchromiumactive_19_v_1_1.png
chrome/icons/abp-19-whitelisted.pngchromiuminactive_19_v_1_1.png
chrome/icons/abp-19-notification-information.pngchromiumquestionmark_19_v_1_1.png
chrome/icons/abp-19-notification-critical.pngchromiumexclamationmark_19_v_1_1.png
chrome/icons/abp-38.pngchromiumactive_38_v_1_1.png
chrome/icons/abp-38-whitelisted.pngchromiuminactive_38_v_1_1.png
chrome/icons/abp-38-notification-information.pngchromiumquestionmark_38_v_1_1.png
chrome/icons/abp-38-notification-critical.pngchromiumexclamationmark_38_v_1_1.png
chrome/icons/abp-48.pngchromium48px-active_v1_2.png
safari/icons/abp-16-notification-information.pngsafariquestionmark_16_v_1_1.png
safari/icons/abp-16-notification-critical.pngsafariexclamationmark_16_v_1_1.png
safari/icons/abp-32-notification-information.pngsafariquestionmark_32_v_1_1.png
safari/icons/abp-32-notification-critical.pngsafariexclamationmark_32_v_1_1.png

The whole list of used icons in the extension can be found here

@palant mentioned in #1534: "All images are overly large, they should go through pngout first."

Attachments (7)

ABP_icon_v_1_0.zip (79.6 KB) - added by sven 5 years ago.
ABP icons v.1.0
ABP_icon_v_1_1.zip (90.9 KB) - added by sven 5 years ago.
ABP icons v.1.1
48px-active_v1_1.png (2.3 KB) - added by christiane 5 years ago.
128px-active_v1_1.png (4.6 KB) - added by christiane 5 years ago.
48px-active_v1_1.png.png (3.4 KB) - added by sven 5 years ago.
48px-active_v1_2.png (3.4 KB) - added by sven 5 years ago.
128px-active_v1_2.png (8.5 KB) - added by sven 5 years ago.

Download all attachments as: .zip

Change History (24)

Changed 5 years ago by sven

ABP icons v.1.0

comment:1 Changed 5 years ago by sven

  • Blocking 1533 added

comment:2 Changed 5 years ago by fhd

  • Priority changed from P2 to P3

comment:3 Changed 5 years ago by fhd

Unsetting ready for now - Sven would still like to go over this with the new designer in January.

comment:4 Changed 5 years ago by sven

  • Description modified (diff)
  • Summary changed from Replace ABP icon with new one (v.1.0) - for Google Chrome to Replace ABP icon with new one (v.1.1) - for Google Chrome

Changed 5 years ago by sven

ABP icons v.1.1

comment:5 Changed 5 years ago by sven

  • Platform changed from Unknown to Chrome

comment:6 Changed 5 years ago by sven

We improved the 16px and the 19px version (v1.1). It's ready to implement now.

comment:7 Changed 5 years ago by sven

  • Cc sebastian trev added

comment:8 Changed 5 years ago by sebastian

  • Summary changed from Replace ABP icon with new one (v.1.1) - for Google Chrome to Replace ABP icon with new one (v.1.1) - for Chrome/Opera/Safari

comment:9 Changed 5 years ago by sebastian

We also need an icon in 48x48 (shown in notifications) and 128x128 (shown when installing the extension and in the list of installed extensions).

Changed 5 years ago by christiane

Changed 5 years ago by christiane

comment:10 Changed 5 years ago by christiane

48x48 and 128x128 active added. see above.

comment:11 Changed 5 years ago by sebastian

  • Cc sebastian removed
  • Owner set to sebastian
  • Ready set

Thanks.

comment:12 Changed 5 years ago by sebastian

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

comment:13 Changed 5 years ago by sebastian

  • Milestone set to Adblock-Plus-for-Chrome-Opera-Safari-next
  • Resolution set to fixed
  • Status changed from reviewing to closed

Changed 5 years ago by sven

Changed 5 years ago by sven

Changed 5 years ago by sven

comment:15 Changed 5 years ago by sven

  • Cc sebastian added
  • Description modified (diff)
  • Ready unset
  • Resolution fixed deleted
  • Status changed from closed to reopened

comment:16 Changed 5 years ago by sven

  • Description modified (diff)

comment:17 Changed 5 years ago by sebastian

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

This issue has already been implemented, as initially specified. So please don't reopen it, for further changes. See #1989 for changing the 48px and 128px icon again.

Note: See TracTickets for help on using tickets.