From 9604680f9f8b25565ba32c06fce8f3f6aac86953 Mon Sep 17 00:00:00 2001 From: Ricky Elrod Date: Wed, 15 Jun 2011 04:52:40 -0400 Subject: [PATCH] Paste sucks when lines are too long. Summary: Paste was cutting off lines when they extended past the width of the screen. Also change a pair of double quotes to single quotes, to follow convention. Test Plan: Tried this in Firefox 4 and Chrome. Reviewers: epriestley, toulouse CC: Differential Revision: 469 --- .../paste/controller/view/PhabricatorPasteViewController.php | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/applications/paste/controller/view/PhabricatorPasteViewController.php b/src/applications/paste/controller/view/PhabricatorPasteViewController.php index 578e5336e0..850fb7842d 100644 --- a/src/applications/paste/controller/view/PhabricatorPasteViewController.php +++ b/src/applications/paste/controller/view/PhabricatorPasteViewController.php @@ -104,7 +104,7 @@ class PhabricatorPasteViewController extends PhabricatorPasteController { $corpus_table = phutil_render_tag( 'table', array( - 'class' => "diffusion-source remarkup-code PhabricatorMonospaced", + 'class' => 'diffusion-source remarkup-code PhabricatorMonospaced', ), implode("\n", $rows)); @@ -127,7 +127,8 @@ class PhabricatorPasteViewController extends PhabricatorPasteController { // And eventually this will highlight a line that you click // like diffusion does. Or maybe allow for line comments // like differential. Either way it will be better than it is now. - $rows[] = ''.$n.''.$line.''; + $rows[] = ''.$n.''. + ''.$line.''; ++$n; }