Styles and "standouts" (and bears, oh my...)

[tyggerjai]

I'm working on a bug in the preview code, and trying to add a "standout" box to preview/entry.bml, which I presume is a markup-vs-content approach to "This information is important and should standout, and we'll let the client decide whether that's bold or flashing red". For most pages, it looks like that would be trivial - there's a lot of "$ret .= <?standout", etc. However, preview/entry.bml seems to have 2 different methods of building pages depending on the styles. One uses the "$ret .= ", and that's fine. The other, however, seems to be a lot more dependent on LJ::S2 code, and that's the bit I'm stuck on.

I have the core2.s2 file now checking whether it's a preview page (thanks to exor674 :), and adding the text I need, but I'm not sure what the appropriate way is to make it "standout" at that level. Clearly for the $ret method, some kind of parsing happens before the final rendering, whereas the s2 in a sense is the final rendering (as far as I can tell :). So can anyone give me a clue as to whether core2.s2 is even the right place to be building the standout, or whether I should add it to $p in preview/entry.bml somewhere, so I can use the ?standout syntax, maybe.

And for the rest of you, hi, I'm Jai :)

jai.
.

Comments

[kareila]

I think putting new code in core2 and/or cgi-bin/LJ/S2.pm is probably the right approach. The preview/entry.bml file is a bit misleading in that it only uses BML for the if ($stylesys == 1) rendering branch. The else branch of that conditional, the S2 branch, is not using BML at all, and the <?standout?> syntax won't work outside of BML.

[tyggerjai]

*nod* That's what I figured. Ok. Is there something definitive I should be using outside of BML to get the same result? 'div class="standout"' doesn't seem effective either :)

[kareila]

There's nothing already available for coding standouts in S2 that I know of. :) Even the standout CSS classes are only defined in the stylesheets for the individual siteschemes. (I see a standout-border class in lj_base-app.css, but no standout class.)

[tyggerjai]

Yeah. I think Afuna and I are going to become very close :)

[tyggerjai]

Out of interest, how do I trigger the $stylesys == 1 section? All the styles I've tried so far seem to be hitting the S2 stuff. Is s1 historical, these days?

[kareila]

$stylesys == 1 means to use siteschemes (tropo, gradation et al) instead of user styles. It's not the same as S1, the old-style LJ layer rendering, which we don't support.

To use the sitescheme for entry and comment pages, uncheck "Show entry pages in my journal style rather than the site layout" under Customize Journal Style: Options: Presentation: Basic Options.

[tyggerjai]

Brilliant, thanks.