Differential D20194 Diff 48202 src/applications/differential/render/DifferentialChangesetRenderer.php
Changeset View
Changeset View
Standalone View
Standalone View
src/applications/differential/render/DifferentialChangesetRenderer.php
| Show First 20 Lines • Show All 355 Lines • ▼ Show 20 Lines | if ($this->getIsTopLevel()) { | ||||
| $notice = $this->renderChangeTypeHeader($force); | $notice = $this->renderChangeTypeHeader($force); | ||||
| } | } | ||||
| $undershield = null; | $undershield = null; | ||||
| if ($this->getIsUndershield()) { | if ($this->getIsUndershield()) { | ||||
| $undershield = $this->renderUndershieldHeader(); | $undershield = $this->renderUndershieldHeader(); | ||||
| } | } | ||||
| $result = $notice.$props.$undershield.$content; | $result = array( | ||||
| $notice, | |||||
| // TODO: Let the user customize their tab width / display style. | $props, | ||||
| // TODO: We should possibly post-process "\r" as well. | $undershield, | ||||
| // TODO: Both these steps should happen earlier. | $content, | ||||
amckinley: Is it intentional that the new whitespace indentation highlighting code triggers here? | |||||
epriestleyAuthorUnsubmitted Done Inline ActionsNo, D20195 should fix this. epriestley: No, D20195 should fix this. | |||||
| $result = str_replace("\t", ' ', $result); | ); | ||||
| return phutil_safe_html($result); | return hsprintf('%s', $result); | ||||
| } | } | ||||
| abstract public function isOneUpRenderer(); | abstract public function isOneUpRenderer(); | ||||
| abstract public function renderTextChange( | abstract public function renderTextChange( | ||||
| $range_start, | $range_start, | ||||
| $range_len, | $range_len, | ||||
| $rows); | $rows); | ||||
| abstract public function renderFileChange( | abstract public function renderFileChange( | ||||
| ▲ Show 20 Lines • Show All 302 Lines • Show Last 20 Lines | |||||
Is it intentional that the new whitespace indentation highlighting code triggers here?