sawyourwolfjunk: (Default)
Kenzi ([personal profile] sawyourwolfjunk) wrote in [community profile] style_system2012-11-13 03:23 pm

Expand issues with transferred layouts

Hi. I'm having a problem with the layouts that I've brought over from LJ. Namely, clicking on the expand link doesn't actually expand the link. It acts like I've hit 'thread'. I'm wondering if there's something I need to add to the code to get it to accept the expand button, so I'm hoping someone can take a look at it, found here, and let me know what/where I need to pop something in.

I've found this to be a problem with most of the layouts I've tried bringing over from LJ. I can customize them to fit DW specs more aptly (since some can get a little wonky with some of the dimensions), but this is the only problem I've had that I can't seem to figure out on my own.

Any help would be greatly appreciated.
Thank you.
chagrined: Marvel comics: zombie!Spider-Man, holding playing cards, saying "Brains?" (brains?)

[personal profile] chagrined 2012-11-17 02:29 am (UTC)(link)
sawyourwolfjunk is correct, you replace calls to expand_link() with print_expand_link(). So any time you find "->expand_link" in your code, replace it with "->print_expand_link".

The second part of my explanation was to split up strings IF changing expand_link to print_expand_link causes you to have problems with the spacing characters (the characters that show up before/after the expand link in your html). To do this, you just put each string on a separate line/expression. In your case:

" " + $c->print_expand_link();

becomes

" ";
$c->print_expand_link();


and

"(" + $comment->print_expand_link() + ") ";

becomes

"(";
$comment->print_expand_link();
") ";


but you only need to do that after you've replaced "->expand_link" with "->print_expand_link", if you have noticed it's botching your spacer characters. Mine did for some reason but this may not be the case for all.
marahmarie: (M In M Forever) (Default)

[personal profile] marahmarie 2012-11-18 04:24 am (UTC)(link)
Yes, yes, the concatenation part I got, but I could never get that far with it because I was using your code ($e->print_expand_link, which does not exist in my theme layer) in place of my code (+ $c->print_expand_link, which can't be replaced with what you posted in your earlier replies on this page), which was throwing compiler errors for me whether I appended print_ or not. Sorry for the confusion.

Oh! And I'm not getting any weird spacing errors yet, but I haven't checked in IE, either. If those turn up I will indeed try concatenating like you said to see if that fixes it. Thanks for your help!
Edited 2012-11-18 04:26 (UTC)