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
F12814036: D9294.id23143.diff
Thu, Mar 28, 1:41 AM
Unknown Object (File)
Thu, Mar 21, 10:08 AM
Unknown Object (File)
Tue, Mar 12, 5:28 PM
Unknown Object (File)
Tue, Mar 5, 2:48 AM
Unknown Object (File)
Feb 18 2024, 9:41 AM
Unknown Object (File)
Feb 6 2024, 8:40 PM
Unknown Object (File)
Feb 4 2024, 3:50 AM
Unknown Object (File)
Jan 25 2024, 6:40 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
Branch
hunk9
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/infrastructure/storage/lisk/PhabricatorLiskDAO.php:193XHP16TODO Comment
Unit
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.

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