Launchpad itself

Syntax highlighting in diffs

Bug #775927 reported by Michael Foord
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Triaged
Low
Unassigned

Bug Description

A nice feature from github pull requests: syntax highlighting of diffs, particularly for merge proposals.

Bug #775926 would probably be a pre-requisite (per file rather than monolithic diffs).

See original description
Henning Eggers (henninge)
Changed in launchpad:
status: New → Triaged
importance: Undecided → Wishlist
Henning Eggers (henninge)
Changed in launchpad:
importance: Wishlist → Low
xaav (xaav)
description: updated
Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

Now that many options are available for this, even client side, could this be added?

Revision history for this message
Colin Watson (cjwatson) wrote :

I'd definitely like to do this, but we need to take some care to ensure that an excessively complex diff can't DoS the webapp, so it's not a trivial thing.

tags: added: code-review lp-code ui
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.