SylvainTI / wwwsqldesigner

Automatically exported from code.google.com/p/wwwsqldesigner
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

Interface zIndex level is too low #83

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Draw more than 105 tables (tip : copy/past XML fragment)
2. Open the save/load dialog

What is the expected output? What do you see instead?
Dialog shall be on top, but appear "under" some tables.

What version of the product are you using? On what operating system?
trunk / Chrome / XP

Please provide any additional information below.
Sorry I noticed this bug early but corrected it locally and forgot about 
it...

This is a CSS issue for property z-index :
 #bar : 100
 #minimap : 100
 #background : 102
 #window : 103
 #rubberband : 99

I added 400 to those number (100 -> 500) but adding more should be better.
Suggestion :
 #bar : 10000
 #minimap : 10000
 #background : 10002
 #window : 10003
 #rubberband : 9999

Someone with 10000 tables in his database might not wish to display it...

For info zIndex is not limited, but the good practice is to limit its value 
to less than 32000.
(unofficial source : http://stackoverflow.com/questions/227738/is-there-an-
upper-limit-to-z-index-values-in-web-browsers)

A better workaround is to define position:absolute or position:relative for 
#area, but it may brings some side-effects.
(official source : http://www.w3.org/TR/CSS21/visuren.html#z-index)

Original issue reported on code.google.com by geompse@gmail.com on 16 Mar 2010 at 7:25

GoogleCodeExporter commented 9 years ago
Acknowledged. I will try experimenting with alternative positions prior to 
increasing
z-index to some insane high levels :)

Original comment by ondrej.zara on 17 Mar 2010 at 8:27

GoogleCodeExporter commented 9 years ago
Fixed in r91. All controls are now grouped in a dedicated layer, whose z-index 
is
dynamically adjusted as necessary.

Original comment by ondrej.zara on 17 Mar 2010 at 12:00