y1feng200156 / dynatree

Automatically exported from code.google.com/p/dynatree
0 stars 0 forks source link

Regression for issue 263 (IE9 overflow:auto causes container div to grow on hover) #470

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
I'd like to ask for this issue to be reopened, since there is a regression. 

Using the current release (jquery.dynatree-1.2.5-all.zip) I have the following 
issue:

The tree div expands to the height of the screen, which breaks the layout of 
the page. It also causes a vertical scroll bar to appear, since the browser 
window is lower than the div. The height is the same until I add more nodes, 
and the div needs even more height. 

The problem can be reproduced with the "4.2.6 Initializing the tree structure 
programmatically" example, using the includes from "3. Quick start" in 
"dynatree-doc.html"

The issue can be observed with IE10 (Windows Vista), Opera 18 (Android), Chrome 
31 (Android, Windows Vista, and Linux 32bit), and the Android 4.04 standard 
browser (AppleWebKit/534.30 (KHTML, like Gecko) Version/4.0 Safari/534.30).

In these browsers mentioned above the div expands as expected if I remove the 
setting *height: 100%* from the IE9 fix.

Opera 12.16 (Linux 32bit), and Firefox 26 (Linux 32bit) tolerate the IE9 fix. 
All the browsers I testes work OK when I remove the *height* property. I 
couldn't test IE9, though. 

Proposed correction: remove the IE9 fix. An alternative fix for the IE9 might 
work with browser detection (a line of jQuery code should do it).

Original issue reported on code.google.com by moo...@wwwendt.de on 4 Jan 2014 at 5:15

GoogleCodeExporter commented 9 years ago
Reported by thomas.goeppel

https://code.google.com/p/dynatree/issues/detail?id=263#c5

Original comment by moo...@wwwendt.de on 4 Jan 2014 at 5:17

GoogleCodeExporter commented 9 years ago
could you send a sample file (or make a fiddle) to reproduce?

Original comment by moo...@wwwendt.de on 4 Jan 2014 at 5:18

GoogleCodeExporter commented 9 years ago
I came across the original issue while investigating a similar problem.  I 
don't know if this will help, but I found an alternative fix to the original 
IE9 problem. Rather than setting "height: 100%", simply set "min-height: 0%" to 
prevent the div from growing unexpectedly.

original source of the fix:
http://blog.brianrichards.net/post/6721471926/ie9-hover-bug-workaround

Original comment by jsgantt.improved on 13 Apr 2014 at 11:34

GoogleCodeExporter commented 9 years ago
thanks!

Original comment by moo...@wwwendt.de on 14 Apr 2014 at 5:26

GoogleCodeExporter commented 9 years ago
This issue was closed by revision r707.

Original comment by moo...@wwwendt.de on 10 May 2014 at 7:29