Missing unified diff view

Bug #253956 reported by Mathias Brodala
4
Affects Status Importance Assigned to Milestone
loggerhead
Fix Released
Medium
Michael Hudson-Doyle

Bug Description

ATM only the side by side view can be used, where there where was also the unified view before the latest Launchpad upgrade.

This looks like a regression to me, thus filing a report for it. Please consider providing a unified view again.

Related branches

Martin Albisetti (beuno)
Changed in loggerhead:
assignee: nobody → beuno
importance: Undecided → Medium
status: New → Confirmed
Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

I wonder if it's possible to turn the side-by-side view into a unified with js rather than rendering the diff twice? It seems to me like it _ought_ to be possible...

Revision history for this message
Martin Albisetti (beuno) wrote :

I can't think of a way to do that with the current layout, although it may me possible somehow.

My guess is that it would be cheaper get all diffs via ajax, and then the user can choose which kind of diff they want (side-by-side, unified, actual changes).

Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

I've attached a branch that implements my idea. It would be great if you could try it out!

Martin Albisetti (beuno)
Changed in loggerhead:
assignee: beuno → mwhudson
status: Confirmed → Triaged
Changed in loggerhead:
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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