Twenty Ten: <pre> tag goes out of content area

[tapan.bojja]

If I enter a <pre> tag in content then it goes out of content area. I think for this css needs to be fix.
Kindly look in to the above issue.

[Moskjis]

The theme you are using, is it Wordpress default one?
If it isn't, then this is the wrong place to create a ticket about it. You need to talk with the theme developer then.

[scribu]

What Moskjis said.

[SergeyBiryukov]

Actually it's the same in Twenty Ten. Attached the patch (just in case).

[dd32]

Confirmed the same thing in Twentyten.

Adding overflow: auto; causes a scrollbar to appear instead of the content continuing off the page.

[nacin]

Want Lance or Ian to weigh in. If they like it, it goes in.

[lancewillett]

Patch looks good with one change: to conform to ​WP CSS standards the overflow property should go before the padding property (alphabetical order). See refreshed patch.

[lancewillett]

Moved property into alpha order

[nacin]

I'm not really a fan of alpha order and I know JohnONolan isn't either. I marked that Codex page as a draft a few months ago as they were never really discussed, and to thus allow for some discussion in the future.

[lancewillett]

Replying to nacin:

I'm not really a fan of alpha order and I know JohnONolan isn't either.

The strongest arguments for alpha order of properties is for readability and ease of debugging—two factors that are prevalent in WordPress code standards in general. Other CSS property ordering patterns are arbitrary and meaningful to the person who created them; alpha order is meaningful universally.

The drafted CSS coding standards should be discussed more, for sure, they are not perfect. But they are a start, and better than nothing. And we will be using them for the default theme's stylesheets.

[JohnONolan]

@lancewillett - There are multiple (valid) standpoints on this issue, but this isn't the place to discuss them :) The CSS coding standards in their current state are very much a working document. If you'd like to come along to UI group meeting - details on ​http://make.wordpress.org/ui - then I'd be happy put this up as an agenda item for discussion.

[lancewillett]

Replying to JohnONolan:

this isn't the place to discuss them

Agreed. I #blamenacin. :)

[SergeyBiryukov]

This is probably fixed, considering the new default theme is Twenty Eleven, which already has overflow: auto; for <pre>. Or Twenty Ten still should be updated?

[SergeyBiryukov]

Related: #17432

[SergeyBiryukov]

Not sure if "WordPress.org" is a proper milestone here, as Twenty Ten is still a bundled theme. Moving to 3.3 for consideration.

[nacin]

Running on WP.com. We'll take that as a go.

[nacin]

In [19300]:

overflow:auto for pre tags in Twenty Ten. props SergeyBiryukov, fixes #16317.