Page MenuHomePhabricator

Substantially support character encodings and "Highlight As" in changesets
ClosedPublic

Authored by epriestley on May 25 2014, 9:10 PM.
Tags
None
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
F13304790: D9294.id.diff
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
Subscribers

Details

Summary

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

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

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.

src/applications/differential/view/DifferentialChangesetListView.php
132

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
src/applications/differential/view/DifferentialChangesetListView.php
132

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).