Page MenuHomePhorge

Teach Commit View about Encoding
ClosedPublic

Authored by avivey on Jul 26 2023, 17:55.

Details

Summary

If the user specifies a text encoding via the "View Options" dropdown, respect this choice.
Ref Q68.

Test Plan

Play with the Encoding button in the view

Diff Detail

Repository
rP Phorge
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

avivey held this revision as a draft.

Just a note from my side: I just applied this patch to our Phorge installation and it does exactly what we need. :)

avivey published this revision for review.Aug 26 2023, 07:46
avivey retitled this revision from teach commit view about encoding to Teach Commit View about Encoding.
avivey edited the test plan for this revision. (Show Details)

Interestingly I tried to open a simple README file and selecting UTF-8.

It said this nonsense message:

This document was converted from UTF-8 to UTF8 for display.

Is this a relevant corner case?

BTW apart from that, this seems very nice, thanks!

This revision is now accepted and ready to land.Aug 26 2023, 10:00
This revision was automatically updated to reflect the committed changes.