Jump to content
Sign in to follow this  
Void Singer

Lithium Changes Break Content

Recommended Posts

Changes to Lithium have broken the ability to post correctly formatted code.

at fault seems to be a behavior change to how "PRE" tags work... originally they were goofed to forc wrapping instead of scrolling within their box, the workaround was simple, but annoying... you aren't allowed by the software to add style attributes to the PRE tags, so one was added to a SPAN inside to prevent line wrapping, reactivating the proper scroll box behavior.

now, adding an encompassing SPAN within the PRE causes the the text to lose all spacing and line returns are converted to spaces. this appears to be a behavior change to the SPAN elements. or the processing of them from the editor.

whatever you guys changed, please put it back, or even better, make PRE tags behave as they are supposed to normally

Share this post


Link to post
Share on other sites

(sorry for all the edits if anyone is subscribed, bad morning.)

I think that the easiest fix for LL will be to change the style sheet.  The trouble we are having is in this sheet, around line 1500, in the section commented "/****** code content */".

 

This part:

#lia-body .lia-content .lia-message-body pre, 	.mceContentBody pre,	#lia-body .lia-content .lia-panel-tooltip pre		{ -ms-word-wrap:sWrap; white-space:-moz-pre-wrap; white-space:-pre-wrap; white-space:-o-pre-wrap; white-space:pre-wrap; }

will work better for us if the "#lia-body .lia-content .lia-message-body pre" and ".mceContentBody pre" rules use white-space:pre and not white-space:pre-wrap. pre-wrap is probably sensible for the tooltips.

Share this post


Link to post
Share on other sites

/me has absolutely no idea what you 2 are talking about, but just came in to see if some of that smartness would rub off on me if I hung around in your thread .lol LMAO.gif

Share this post


Link to post
Share on other sites

(Psst, Peewee - they have cool tools for formatting scripts when they paste them into posts.  Now their toys are broken.  *whispers* I have no idea what they're talking about either, beyond that.  Couldn't understand the thread where they were telling us all how to use the tools)

Share this post


Link to post
Share on other sites


Void Singer wrote:

Changes to Lithium have broken the ability to post correctly formatted code.

at fault seems to be a behavior change to how "PRE" tags work... originally they were goofed to forc wrapping instead of scrolling within their box, the workaround was simple, but annoying... you aren't allowed by the software to add style attributes to the PRE tags, so one was added to a SPAN inside to prevent line wrapping, reactivating the proper scroll box behavior.

now, adding an encompassing SPAN within the PRE causes the the text to lose all spacing and line returns are converted to spaces. this appears to be a behavior change to the SPAN elements. or the processing of them from the editor.

whatever you guys changed, please put it back, or even better, make PRE tags behave as they are supposed to normally

I will pass this on for review!  Thanks for the feedback. 

Share this post


Link to post
Share on other sites

Thanks Lexie =)

@Pete:
it's not so much about the code hilighter, but about the goofy wrapping behavior that makes code posting wrap, making it harder for inexperienced users to understand (and a little harder for experienced coders to read, but that's just mostly just an annoyance for us). the hilighter uses a workaround to stop that, but a change to the forums causes that to collapse indents and remove line breaks (something that shouldn't be happening ever)

Share this post


Link to post
Share on other sites

Addendum:
I didn't catch it before, but the font was also changed from fixed width, and that is also something that needs repaired.

Most preformatted text (not just code) relies of having a fixed width font for various reasons, and with the changes to span behavior we cannot repair this without a great amount of effort and up to 4 times the byte count used normally.

Share this post


Link to post
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Sign in to follow this  

×
×
  • Create New...