Status: Accepted
Owner: ----
Labels: Type-Defect Priority-Medium Component-ChangeDescriptions

New issue 3909 by ste...@smacleod.ca: Change Description diff highlighting for changed part of line is incorrect
https://code.google.com/p/reviewboard/issues/detail?id=3909

What version are you running?
2.0.17

What's the URL of the page containing the problem?
/r/<rr-id>/

It appears the highlighting for which part of a line changed in change description diffs can be incorrect. A screenshot of the issue can be found at
https://bug1179930.bugzilla.mozilla.org/attachment.cgi?id=8629020

This was filed against MozReview as https://bugzilla.mozilla.org/show_bug.cgi?id=1179930 with the relavant review request being https://reviewboard.mozilla.org/r/12289/


The problematic changedescription contains the following:

{u'diff': {u'added': [[u'Diff r2', u'/r/12289/diff/2/', 25957]]}, u'description': {u'new': [u'Bug 1178892 - Split the profiler into Core & Gecko files and break apart monolithic headers. r=mstange'], u'old': [u'Bug 1178892 - Split the profiler into Core & Gecko files and break apart monolithic headers. r?mstange']}, u'summary': {u'new': [u'Bug 1178892 - Split the profiler into Core & Gecko files and break apart monolithic headers. r=mstange'], u'old': [u'Bug 1178892 - Split the profiler into Core & Gecko files and break apart monolithic headers. r?mstange']}}


--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.

Reply via email to