Opened on 11/25/2014 at 04:49:23 PM

Closed on 11/26/2014 at 11:39:27 AM

#1607 closed defect (fixed)

"Block element" injects overlays with wrong z-index

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

http://codereview.adblockplus.org/4605237906112512

Description (last modified by sebastian)

Adapted from https://www.passbrains.com/dashboard/view-ticket.php?ticket_no=AOP-119

Environment

Windows + Vista 64bit + Opera + English
ABP version Adblock Plus Version 1.8.7.1253

How to reproduce

  1. Install ABP extension on Opera.
  1. Open website

http://www.flipkart.com/
 
3.click on "signup".
A popup window will appear.

  1. Try to block Facebook, Google, X elements.
  2. Observe.

Observed behaviour

Not able to block Facebook, Google, X elements, because back ground elements getting selected to block.
See attached screen shot.

Expected behaviour

Should be able to block Facebook, Google, X elements.

Attachments (1)

3668_1416060227_flipkart1.jpg (106.6 KB) - added by passbrains on 11/25/2014 at 04:49:26 PM.

Download all attachments as: .zip

Change History (5)

Changed on 11/25/2014 at 04:49:26 PM by passbrains

comment:1 Changed on 11/25/2014 at 05:48:49 PM by sebastian

  • Component changed from Unknown to Platform
  • Description modified (diff)
  • Owner set to sebastian
  • Platform changed from Opera to Unknown
  • Priority changed from Unknown to P4
  • Ready set

comment:2 Changed on 11/25/2014 at 05:52:37 PM by sebastian

  • Status changed from new to reviewing
  • Summary changed from Not able to block Facebook,Google elements. to "Block element" injects overlays with wrong z-index

That is because the overlay we inject for the image below the login dialog, uses a higher z-index than that dialog. The overlays should always use the same z-index as element they were created for, to make sure that they are positioned over that element, but below other elements with higher z-index.

comment:3 Changed on 11/25/2014 at 05:54:15 PM by sebastian

  • Review URL(s) modified (diff)

comment:4 Changed on 11/26/2014 at 11:39:27 AM 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.