Tree: hardcoded 19px padding breaking my site's layout

[Scott Moxham]

v1.3.0 has added some inline paddingLeft of 19px into the dijit.tree nodes for indentation. This was not in 1.2.3 and it breaks the layout of my site.

This indent really needs to be in CSS instead of inline so that it can be easily overridden.

[bill]

Yup, I agree, I added a comment for this in [16088] but forgot to file a ticket for it (or fix it). Thanks for the reminder.

[travis.haagen]

Can someone please describe how to fix this manually. For some reason, I have had difficulty finding where the 19px indentation is hard-coded. Thanks.

[bill]

I'm going to work on this soon but to change it manually just search for the string "19" in Tree.js.

[bill]

(In [17543]) Make indent for Tree's nodes adjustable via CSS.

Since dojo.marginBox() doesn't work reliably until the Tree's DOM node is inserted into the document and is visible, the CSS width is read when Tree.resize() is called. If Tree.resize() isn't called then it defaults to 19px, as before (for backwards compatibility).

Fixes #9078 !strict.

[Nathan Toone]

(In [17554]) Refs #9078 - when resizing a tree that does not show root, the whole tree gets indented. Honor the showRoot instead !strict

[bill]

In [28846]:

If the Tree is hidden during initial resize for some reason, then use default 19px indent rather than 0px indent.

Refs #9078 !strict.