Opened on 12/08/2014 at 10:49:55 AM

Closed on 12/11/2014 at 02:50:59 PM

#1671 closed change (fixed)

Remove "Hide placeholders" option from the UI on Chrome/Opera/Safari

Reported by: sebastian Assignee: sebastian
Priority: P3 Milestone: Adblock-Plus-1.8.9-for-Chrome-Opera-Safari
Module: Platform Keywords:
Cc: trev, greiner Blocked By:
Blocking: Platform: Unknown
Ready: yes Confidential: no
Tester: Verified working: no
Review URL(s):

http://codereview.adblockplus.org/6201561285918720

Description

Background

The effect of that option is limitted and confusing, since it doesn't have any effect on element hiding. Also we assume only very few users using this option. And correctly considering that option makes things more complicated in some cases (e.g. with the "Block Element" functionality).

But for now, we only want to remove that option from the UI. So if users complain about that change, they can disable element collapsing again on the console.

What to change

Remove the "Hide placeholders" option from the options page and reset the option, always collapsing blocked elements. However, keep the functionlity in there for now.

Attachments (0)

Change History (3)

comment:1 Changed on 12/08/2014 at 10:50:56 AM by sebastian

See #1670, for respective Firefox issue.

comment:2 Changed on 12/08/2014 at 06:14:21 PM by sebastian

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

comment:3 Changed on 12/11/2014 at 02:50:59 PM by sebastian

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

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.