recaptime.dev / phorge
@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.) hq.recaptime.dev/wiki/Phorge
phorge phabricator
1
fork

Configure Feed

Select the types of activity you want to include in your feed.

Fix tab indentation missing in Diffusion

This seems to be a specific of how browsers are dealing with
spaces/tabs. Multiple spaces works just fine, but multiple
tabs were treating as a single space which breaks indentation.

Now made it so tabs are replaced with 4 spaces. Not ideal but
still better than fully unreadable code. This also matches to
how differential is handling tabs.

Ref T2495. See: <https://github.com/facebook/phabricator/issues/487>

Reviewed by: epriestley

authored by

Sergey Sharybin and committed by
epriestley 035c79e7 01a80af9

+1 -1
+1 -1
src/applications/diffusion/controller/DiffusionBrowseFileController.php
··· 774 774 "\xE2\x80\x8B", 775 775 776 776 // TODO: [HTML] Not ideal. 777 - phutil_safe_html($line['data']), 777 + phutil_safe_html(str_replace("\t", ' ', $line['data'])), 778 778 )); 779 779 780 780 $rows[] = phutil_tag(