Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 5859059588661248: Issue 2077 - Inject overlays with highest possible z-index (Closed)

Created:
April 8, 2015, 1:28 p.m. by Sebastian Noack
Modified:
April 8, 2015, 2:54 p.m.
Reviewers:
kzar
CC:
Wladimir Palant
Visibility:
Public.

Description

Issue 2077 - Inject overlays with highest possible z-index

Patch Set 1 #

Total comments: 3
Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -15 lines) Patch
M include.postload.js View 3 chunks +1 line, -15 lines 3 comments Download

Messages

Total messages: 5
Sebastian Noack
April 8, 2015, 1:29 p.m. (2015-04-08 13:29:05 UTC) #1
kzar
LGTM
April 8, 2015, 2:05 p.m. (2015-04-08 14:05:52 UTC) #2
kzar
http://codereview.adblockplus.org/5859059588661248/diff/5629499534213120/include.postload.js File include.postload.js (right): http://codereview.adblockplus.org/5859059588661248/diff/5629499534213120/include.postload.js#newcode257 include.postload.js:257: overlay.style.zIndex = 0x7FFFFFFE; Hang on, shouldn't that be 0x7FFFFFFF?
April 8, 2015, 2:12 p.m. (2015-04-08 14:12:21 UTC) #3
Sebastian Noack
http://codereview.adblockplus.org/5859059588661248/diff/5629499534213120/include.postload.js File include.postload.js (right): http://codereview.adblockplus.org/5859059588661248/diff/5629499534213120/include.postload.js#newcode257 include.postload.js:257: overlay.style.zIndex = 0x7FFFFFFE; On 2015/04/08 14:12:22, kzar wrote: > ...
April 8, 2015, 2:32 p.m. (2015-04-08 14:32:45 UTC) #4
kzar
April 8, 2015, 2:33 p.m. (2015-04-08 14:33:40 UTC) #5
LGTM

http://codereview.adblockplus.org/5859059588661248/diff/5629499534213120/incl...
File include.postload.js (right):

http://codereview.adblockplus.org/5859059588661248/diff/5629499534213120/incl...
include.postload.js:257: overlay.style.zIndex = 0x7FFFFFFE;
On 2015/04/08 14:32:45, Sebastian Noack wrote:
> On 2015/04/08 14:12:22, kzar wrote:
> > Hang on, shouldn't that be 0x7FFFFFFF?
> 
> No, 0x7FFFFFFF is the z-index of the dialog which is supposed to cover the
> overlays highlighting selected/matching elements. So we have to use a lower
> z-index here.

Gotya.

Powered by Google App Engine
This is Rietveld