Making WordPress.org

Opened 8 years ago

Closed 4 years ago

Last modified 19 months ago

#333 closed defect (fixed)

Trac: Source View Line Wrapping

Reported by: netweb Owned by: obenland
Milestone: Priority: normal
Component: Trac Keywords: has-patch


When viewing a repo's source line wrapping ain't quite right.

eg. https://bbpress.trac.wordpress.org/browser/trunk/templates/default/bbpress/form-topic-merge.php#L40

The full line of code per the above is:

40 <?php if ( bbp_has_topics( array( 'show_stickies' => false, 'post_parent' => bbp_get_topic_forum_id( bbp_get_topic_id() ), 'post__not_in' => array( bbp_get_topic_id() ) ) ) ) : ?>

It is displayed across three lines as:

php if ( bbp_has_topics( array( 'show_stickies' => false, 'post_parent' => bbp_get_topic_forum_id( bbp_get_topic_id() ), 'post

If you remove overflow: hidden (bbpress.trac.wordpress.org/chrome/common/css/code.css#Line 69) you can see the remaining code 'off screen'

Attachments (2)

333.png (22.3 KB) - added by netweb 8 years ago.
333.diff (512 bytes) - added by xkon 4 years ago.
Span wrapping

Download all attachments as: .zip

Change History (11)

8 years ago

#1 @nacin
7 years ago

  • Resolution set to worksforme
  • Status changed from new to closed

I can't reproduce this. Whenever I see long lines, they wrap. Please reopen with a new line demonstrating this and tag it to a revision, as that file no longer exists for trunk.

#2 @netweb
7 years ago

  • Resolution worksforme deleted
  • Status changed from closed to reopened

Here's that same file tied to a revision, which is still not wrapping as expected ;)


#3 @Otto42
7 years ago

Wraps for me. What are you seeing?


Version 0, edited 7 years ago by Otto42 (next)

#4 @xkon
4 years ago

  • Keywords has-patch added

Although it's an old topic you can easily fix the wrapping + the overflow problem by changing the spans that are generated. ( 333.diff )

By viewing the source I think I have the correct .css file. This should do the trick it will just wrap everything as they hit the end of the table.

table.code td span {
	display: inline-block;

Best regards,

4 years ago

Span wrapping

#5 @birgire
4 years ago

Here are other problematic code lines, number 5311 and 5312:


The part is inside <td class="r"><ins>...</ins></td>.

#6 @obenland
4 years ago

  • Owner set to obenland
  • Resolution set to fixed
  • Status changed from reopened to closed

In 6524:

Trac: Wrap overflowing lines in code tables.

Props xkon, netweb.
Fixes #333.

This ticket was mentioned in Slack in #meta-wordcamp by iandunn. View the logs.

22 months ago

This ticket was mentioned in Slack in #meta-wordcamp by sippis. View the logs.

22 months ago

This ticket was mentioned in Slack in #meta-wordcamp by sippis. View the logs.

19 months ago

Note: See TracTickets for help on using tickets.