Opened on 07/20/2016 at 10:54:45 AM

Closed on 07/21/2016 at 04:16:26 PM

Last modified on 07/26/2016 at 09:53:26 AM

#4256 closed change (fixed)

Element Hiding Helper should use larger ABP icon on high-DPI screens

Reported by: trev Assignee: trev
Priority: P3 Milestone: Element-Hiding-Helper-1.3.9
Module: Extensions-for-Adblock-Plus Keywords:
Cc: Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Scheer Verified working: yes
Review URL(s):

https://codereview.adblockplus.org/29348059/

Description

Background

Element Hiding Helper will currently use the 16x16 icon in the Inspector panel of Developer Tools. On high-DPI screens this is suboptimal.

What to change

Use larger icon depending on pixel density.

Attachments (4)

EHH New 150%.PNG (20.8 KB) - added by scheer on 07/26/2016 at 09:53:42 AM.
EHH New 200%.PNG (62.0 KB) - added by scheer on 07/26/2016 at 09:53:49 AM.
EHH Old 150%.PNG (18.5 KB) - added by scheer on 07/26/2016 at 09:53:58 AM.
EHH Old 200%.PNG (50.6 KB) - added by scheer on 07/26/2016 at 09:54:05 AM.

Download all attachments as: .zip

Change History (8)

comment:1 Changed on 07/20/2016 at 10:58:04 AM by trev

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

comment:2 Changed on 07/21/2016 at 04:15:22 PM by abpbot

comment:3 Changed on 07/21/2016 at 04:16:26 PM by trev

  • Milestone set to Element-Hiding-Helper-next
  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:4 Changed on 07/26/2016 at 09:53:26 AM by scheer

  • Tester changed from Unknown to Scheer
  • Verified working set
  • The EHH icon now scales correctly dependent on the DPI scale set. A huge difference can be noted between the versions as depicted in the images attached.

EHH New - 1.3.8.536-beta
EHH Current - 1.3.8 (Live Version)
Firefox 45, 48
Windows 10 64 Bit

Changed on 07/26/2016 at 09:53:42 AM by scheer

Changed on 07/26/2016 at 09:53:49 AM by scheer

Changed on 07/26/2016 at 09:53:58 AM by scheer

Changed on 07/26/2016 at 09:54:05 AM by scheer

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 trev.
 
Note: See TracTickets for help on using tickets.