Page MenuHomePhabricator

Substantially support character encodings and "Highlight As" in changesets

Authored by epriestley on May 25 2014, 9:10 PM.
Referenced Files
F13323429: D9294.diff
Fri, Jun 14, 5:58 PM
F13317084: D9294.diff
Thu, Jun 13, 9:02 AM
F13315951: D9294.diff
Wed, Jun 12, 8:38 PM
Sat, Jun 8, 12:58 PM
F13302679: D9294.diff
Sat, Jun 8, 5:33 AM
F13286218: D9294.diff
Tue, Jun 4, 6:50 AM
F13273108: D9294.diff
Fri, May 31, 1:35 AM
F13272718: D9294.id23143.diff
Thu, May 30, 6:48 PM



Ref T5179. Ref T4045. Ref T832. We can now write non-utf8 hunks into the database, so try to do more reasonable things with them in the UI.

Test Plan

(See screenshots...)

Diff Detail

rP Phabricator
Lint Passed
Advicesrc/infrastructure/storage/lisk/PhabricatorLiskDAO.php:193XHP16TODO Comment
Tests Passed
Build Status
Buildable 705
Build 705: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley retitled this revision from to Substantially support character encodings and "Highlight As" in changesets.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.

Here's a Shift-JIS diff being rendered correctly (we guess the encoding properly):

Screen_Shot_2014-05-25_at_1.31.56_PM.png (332×1 px, 64 KB)

Here's the menu for changing encoding/highlight options:

Screen_Shot_2014-05-25_at_1.42.24_PM.png (319×331 px, 26 KB)

Here's the same diff rendered explicitly as Shift-JIS (via the menu):

Screen_Shot_2014-05-25_at_1.32.05_PM.png (333×1 px, 65 KB)

Here's the diff rendering with the wrong encoding (also via the menu):

Screen_Shot_2014-05-25_at_1.32.19_PM.png (286×1 px, 43 KB)

Here's rainbow highlighting:

Screen_Shot_2014-05-25_at_1.32.47_PM.png (427×1 px, 82 KB)

btrahan edited edge metadata.

Whoops, sorry, I had looked at this at length weeks ago and didn't quite finish for some reason.


Is "unified" the colloquial term here? (Does a colloquial term exist?) I'd probably go with "Inline" myself here.

This revision is now accepted and ready to land.Jun 20 2014, 5:21 PM

Unified comes from diff --unified, which I think is really talking about the diff format rather than its single-columnness, but I've seen at least one or two users use this term in the same way as here.

If there's any confusion, we could call this something else. (I worry "inline" might cause some confusion with inline comments, though.)

epriestley updated this revision to Diff 23143.

Closed by commit rPb20884a842a6 (authored by @epriestley).