Red Hat Bugzilla – Bug 980695
Diff is highlighting changes in wrong order
Last modified: 2014-08-04 18:27:35 EDT
Description of problem:
The diff tool systems to be doing the change highlighting in the wrong order.
The order of the revisions displayed on-screen for left/right is correct (i.e. the newer revision is on the left), but the change highlighting seems to show the revision on the left as being the superseded one.
i.e. something added to the newer revision (displayed on the left), is shown in red with a strikethrough, and something that was replaced in the older revision (displayed on the right), is shown with a bluey background.
The highlighting probably should be the other way around?
Bug is present in the Next-UI Build 201307011553
arrg, the first sentence in the description was supposed to be:
"The diff tool seems to be doing the change highlighting in the wrong order."
Fixed in 201307091606
Diff order is reversed, so the colours make more sense.
When QAing bug fixes, please make sure the build you are using is equal to or higher than the build identified above. Be aware that this fix may have only been applied in PressGang Next at this time, or it may not be currently applied in any production system, in which case it can only be verified at a later date.
Verified that it is working now, although the sides of the displayed revisions have now changed (i.e. the older revision is now on the left, and the newer revision is now on the right).
As this it looks like this is the intended usage of mergely, It might be worth mentioning this change in the release notes.
I believe what we probably should do is look at adding headers to the diff view so that it will display the revision number, as that would make things a lot clearer. As such I've created BZ#991295 to deal with that.