Bug 756264

Summary:	Highlight differences compared to Translation Memory matches
Product:	[Retired] Zanata	Reporter:	Sean Flanigan <sflaniga>
Component:	Component-UI	Assignee:	Alex Eng <aeng>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Carlos Munoz <camunoz>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	1.5-SNAPSHOT	CC:	camunoz, runab, zanata-bugs
Target Milestone:	Sprint-30	Keywords:	UserStory
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
URL:	https://community.rallydev.com/slm/rally.sp#/3926933431d/detail/userstory/4764716391
Whiteboard:
Fixed In Version:	1.5-SNAPSHOT	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2012-04-23 04:32:41 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	756230

Description Sean Flanigan 2011-11-23 04:15:41 UTC

User story: As a translator I want the Suggestions from the Translation Memory to highlight the differences with the current string so that I can view the part of the string which is an exact match to the string


References:

http://stackoverflow.com/questions/132478/how-to-perform-string-diffs-in-java
http://code.google.com/p/google-diff-match-patch/

Comment 1 Sean Flanigan 2011-12-06 00:41:56 UTC

I've done some of the work here: https://github.com/zanata/zanata/tree/rhbz756264

The colour scheme still needs some work.  I think we need an option (easily accessible, not buried in the options menu) to hide the deleted/missing text, which would switch from a diff view to highlighting the matching text.

Comment 2 Sean Flanigan 2011-12-16 06:31:32 UTC

Alex, could you please merge my changes into master?

Comment 3 Alex Eng 2011-12-19 01:59:14 UTC

Merged into Master. See https://github.com/zanata/zanata/commit/47fbc1d181848431064cb4f12427619faf1745d9

Comment 4 Carlos Munoz 2012-01-06 02:29:45 UTC

Verified against 1.5-SNAPSHOT.

Comment 5 Runa Bhattacharjee 2012-01-13 04:31:09 UTC

Verified. Can the colour be enhanced slightly to highlight the contrast better? Thanks.

Comment 6 Sean Flanigan 2012-01-30 06:25:58 UTC

How about these Lokalize colours then? 
http://docs.kde.org/stable/en/kdesdk/lokalize/tm.html

That's #99CCFF and #FF9999

Comment 7 Runa Bhattacharjee 2012-01-30 12:29:54 UTC

These would be better. Thanks.

Comment 8 Sean Flanigan 2012-01-31 23:52:11 UTC

Colour change implemented in https://github.com/zanata/zanata/commit/fdd937a6ede956abf17d0650a4694d9c673a713c (master branch)