jquery.treeTable.js expandable tree margin-left is to far to the left

Patch attached. It fixes the wrong table allignment and moves the image inside the table.

Needs screenshots on Bartik, Garland, and Seven themes in D7.

@thekevinday: I guess he mean with the patch applied :-)

I can confirm seven works well with the patch

Sorry, I should have included screenshots should include a leaf expanded ... last time we tried this simple fix it looked like crap.

For me this looks good in seven.

Not really. The [user:created:custom:?] token should be indented past [user:created:custom] since it's the next level.

The structure Looks completly the same without the patch. There is no [user:created:custom]. I'm confused...

I have also tested the patch on D6 with latest tokens DEV and it works as it should with the patch applied. UI looks good with patch, but had the same issues like D7 without the patch. Tested with my YAML theme. The patch above applies also to D6-DEV with an offset.

I've got a patch that addresses the problem by adding a class on the first column and adding some padding. On it's own, the negative margin in the tree-table makes sense, so I found it better to address with the table.

Here's what it should look like after (Chrome / OSX):

#11: token_indent-961130-11.patch queued for re-testing.

Hrm, the most recent patch doesn't seem to fix the fact that the table itself is indented to the left, which is mostly what we're trying to fix as well.

You should see this duplicate issue from tarekdj #1423022: margin problem with replacement patterns table

Including the below change in the above patch fixes it for me.

 .token-tree {
   font-size: 0.85em;
-  margin-left: 19px;
 }

Here is a how I am doing it. It includes the patch from #11 and part of the patch from #1423022: margin problem with replacement patterns table

Attaching proper patch format against latest dev.

With patch in #20 we are unable to expand the token group in available Tokens table.
So, updated the patch.

Interdiff :
added back $row['id'] = _token_clean_css_identifier($token);

but $row['id'] = _token_clean_css_identifier($token); is not in the patch. Could you re-roll the patch?

//Naveen

@naveenvalecha: It's already there.

$row['id'] = _token_clean_css_identifier($token); Is already in the code at line 201 and was removed in patch #20
So, I removed that exclusion in #21

Hope it clears the confusion.

The 2 year old patch in #21 to token.css applied cleanly to the latest token 7.x-1.x-dev and if still applicable needs to be reviewed.