Issue 49: css issues in strict mode
Status:  Duplicate
Owner:
Closed:  Nov 2008
Reported by gabriell...@gmail.com, Oct 29, 2008
What steps will reproduce the problem?

1. create a dynatree using <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML
4.01//EN" "http://www.w3.org/TR/html4/strict.dtd">

What is the expected output? What do you see instead?

The 'T' shaped dotted line images (ltL_nes.gif) do not connect (they have
some vertical spacing in between each one.

What version of the product are you using?

jquery.dynatree.js 0.3, beta build 2008-09-14 

 
On what operating system and browser?

Mac OSX FF 3 & Safari 3


Please provide any additional information below.

In strict mode images are not treated as inline elements.  The get some
extra margin added to the bottom of them.

Creating a custom style sheet can fix this
(tested in FF, Safari and IE6)

add the following attributes to .ui-dynatree-container img
display: block; /*removes bottom margin from images*/
float: left;    /*makes images display inline*/

and add the following rule
/*clear the float left from above */
div.ui-dynatree-container div div {
    clear:both;
}

Nov 1, 2008
Project Member #1 moo...@wwwendt.de
(No comment was entered for this change.)
Status: Accepted
Owner: moo...@wwWendt.de
Labels: Milestone-Release0.3
Nov 1, 2008
Project Member #2 moo...@wwwendt.de
the proposed solution seemed to wor (FF3 and IE7) with this exception:
I used
  div.ui-dynatree-container div {
instead of 
  div.ui-dynatree-container div div {

>> r72
Needs to be verified on safary, ie6, ff2

Status: Fixed
Nov 1, 2008
Project Member #3 moo...@wwwendt.de
Had to re-open this!

The CSS changes caused a browser crash in IE 7.0.6001.18000 on Vista
when using the [Left] key to close an expanded node.

It's reproducible and only occurs, when the "float: ..." line is present.

.ui-dynatree-container img
{
	[...]
	display: block; /* Removes bottom margin from images */
	float: left;    /* Makes images display inline */
}

/*clear the float left from above */
div.ui-dynatree-container div  
{
    clear: both; 
}


Status: Accepted
Nov 15, 2008
Project Member #4 moo...@wwwendt.de
Maybe fixed with  issue #55 . Still needsto be verified 
Nov 30, 2008
Project Member #5 moo...@wwwendt.de
Closing this.
Please provide feedback, if the current css doesn't work for you.
Status: Duplicate