Jump to content

Lithium Changes Break Content


Void Singer
 Share

You are about to reply to a thread that has been inactive for 4155 days.

Please take a moment to consider if this thread is worth bumping.

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

Link to comment
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.

Link to comment
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)

Link to comment
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. 

Link to comment
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)

Link to comment
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.

Link to comment
Share on other sites

You are about to reply to a thread that has been inactive for 4155 days.

Please take a moment to consider if this thread is worth bumping.

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
 Share

×
×
  • Create New...