Fix overlay.css so close button is on top of the page.

Fix overlay.css so close button is on top of the page.

BUG=168424
TEST=manual
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=175940

[achuithb, 2013-01-09 00:31:14]

This may be a bug in the css:
http://code.google.com/codesearch#OAMlx_jo-ck/src/chrome/browser/resources/sh...

If I leave out the h1 element, the close button stops responding (hover and
click do nothing).

This fix works, but perhaps there's something better we can do?

[Dan Beam, 2013-01-09 02:40:49]

how about:

  .overlay .page {
    z-index: 0;
  }

  .close-button {
    z-index: 1;
  }

?

that seems to fix your problem globally without adding extra space at the top of
some dialog for now reason.

[achuithb, 2013-01-09 08:51:02]

Dan: I've done as you suggest, and it works. PTAL.

Could you explain why it was broken?

[achuithb, 2013-01-09 18:38:19]

On 2013/01/09 08:51:02, achuith.bhandarkar wrote:
> Dan: I've done as you suggest, and it works. PTAL.
> 
> Could you explain why it was broken?

ping!I need to merge this into m25 before 7p today

[xiyuan, 2013-01-09 18:39:05]

LGTM

[Dan Beam, 2013-01-10 00:12:43]

lgtm