Bug 756264 - Highlight differences compared to Translation Memory matches
Summary: Highlight differences compared to Translation Memory matches
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Zanata
Classification: Retired
Component: Component-UI
Version: 1.5-SNAPSHOT
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: Sprint-30
: ---
Assignee: Alex Eng
QA Contact: Carlos Munoz
URL: https://community.rallydev.com/slm/ra...
Whiteboard:
Keywords: UserStory
Depends On:
Blocks: 756230
TreeView+ depends on / blocked
 
Reported: 2011-11-23 04:15 UTC by Sean Flanigan
Modified: 2012-04-23 04:32 UTC (History)
3 users (show)

(edit)
Clone Of:
(edit)
Last Closed: 2012-04-23 04:32:41 UTC


Attachments (Terms of Use)

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)


Note You need to log in before you can comment on or make changes to this bug.