#333 closed defect (bug) (fixed)
Trac: Source View Line Wrapping
Reported by: | netweb | Owned by: | obenland |
---|---|---|---|
Milestone: | Priority: | normal | |
Component: | Trac | Keywords: | has-patch |
Cc: |
Description
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)
Change History (11)
#2
@
10 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 ;)
#4
@
7 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,
Konstantinos
#5
@
7 years ago
Here are other problematic code lines, number 5311 and 5312:
https://core.trac.wordpress.org/attachment/ticket/35293/35293.4.diff
The part is inside <td class="r"><ins>...</ins></td>
.
#6
@
7 years ago
- Owner set to obenland
- Resolution set to fixed
- Status changed from reopened to closed
In 6524:
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.