[Bug 56403] Use DOM append instead of z-index to position lightbox

2014-06-26 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=56403

Tisza Gergő gti...@wikimedia.org changed:

   What|Removed |Added

   See Also||https://bugzilla.wikimedia.
   ||org/show_bug.cgi?id=60405

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 56403] Use DOM append instead of z-index to position lightbox

2014-02-21 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=56403

--- Comment #9 from Gilles Dubuc gdu...@wikimedia.org ---
Yep, display: none is being applied already (could be taken a little bit
further, though, by hiding all children except the mmv elements, currently it
uses a whitelist). I think we can safely get rid of z-index. Any objections?

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 56403] Use DOM append instead of z-index to position lightbox

2014-02-20 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=56403

Gilles Dubuc gdu...@wikimedia.org changed:

   What|Removed |Added

 CC||gdu...@wikimedia.org

--- Comment #3 from Gilles Dubuc gdu...@wikimedia.org ---
What else is using z-index right now, that would prevent us from moving to
append? Isn't it risky to bet on the fact that all other extensions will play
nice and not use z-index?

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 56403] Use DOM append instead of z-index to position lightbox

2014-02-20 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=56403

--- Comment #4 from Mark Holmquist mtrac...@member.fsf.org ---
The backdrop, wrapper, and maybe the file usage dialog(s) are all using various
values for z-index. Apparently the buttons are, too.

It shouldn'

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 56403] Use DOM append instead of z-index to position lightbox

2014-02-20 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=56403

--- Comment #5 from Mark Holmquist mtrac...@member.fsf.org ---
...t be too hard to fix this, though, it's just a matter of making sure
everything works well.

Honestly we could probably display:none the rest of the DOM while the lightbox
is open.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 56403] Use DOM append instead of z-index to position lightbox

2014-02-20 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=56403

--- Comment #6 from Mark Holmquist mtrac...@member.fsf.org ---
...oh, we do already. We don't need z-index for that, then.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 56403] Use DOM append instead of z-index to position lightbox

2014-02-20 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=56403

--- Comment #7 from Tisza Gergő gti...@wikimedia.org ---
We are appending to the end of the body (maybe that was different when this bug
was opened), but I see no good reason not to have a z-index as well. Any
administrator on any wiki can put a z-index rule into MediaWiki:Common.css and
cause us problems. Is there ever a valid reason to position something above the
lightbox? (Notifications, maybe?)

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 56403] Use DOM append instead of z-index to position lightbox

2014-02-20 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=56403

--- Comment #8 from Tisza Gergő gti...@wikimedia.org ---
...although display:none works as well.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 56403] Use DOM append instead of z-index to position lightbox

2013-11-19 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=56403

--- Comment #2 from Mark Holmquist mtrac...@member.fsf.org ---
I guess the idea here is that we should *all* be using the append-to-DOM method
as opposed to the z-index method, because the z-index method is DUMB.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 56403] Use DOM append instead of z-index to position lightbox

2013-11-04 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=56403

Mark Holmquist mtrac...@member.fsf.org changed:

   What|Removed |Added

 CC||krinklem...@gmail.com
 Blocks||700

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 56403] Use DOM append instead of z-index to position lightbox

2013-10-31 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=56403

Tisza Gergő gti...@gmail.com changed:

   What|Removed |Added

 CC||gti...@gmail.com

--- Comment #1 from Tisza Gergő gti...@gmail.com ---
I don't think so - anything with a z-index would still be above the lightbox.

Also, AFAIK z-index is reliable enough as long as you are using it on direct
children of the body - the IE6 bug is related to z-indexed elements having
non-static parents.

The z-index on .mlb-fullscreen-div is pointless, however - its parent also has
a z-index so that one will determine how it is placed.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 56403] Use DOM append instead of z-index to position lightbox

2013-10-31 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=56403

Fabrice Florin fflo...@wikimedia.org changed:

   What|Removed |Added

   Priority|Unprioritized   |Low

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l