GNOME Bugzilla – Bug 566060
Integrate to podiff
Last modified: 2012-10-17 14:38:00 UTC
It'd be great to have links to a diff file (made with podiff, since else it's useless) when you attach a new po file. Ideally, you'd have: + podiff with the current translation in svn (the one that can be downloaded from damned-lies) + podiff with the previously attached translations (for example, when you do a review a translation and change a few strings) This feature is already available in Transifex :)
*** Bug 566558 has been marked as a duplicate of this bug. ***
*** Bug 567022 has been marked as a duplicate of this bug. ***
This bug report has some overlap with #550208.
Preliminary work has been committed in rev. 1357 (not in production yet). There is still improvements ahead regarding po specific syntax. I just used standard Python difflib.HtmlDiff functionality for now.
Created attachment 126668 [details] [review] Generating a patch-like diff instead of html table I've attached a patch that uses bzr's bzrlib.patiencediff to generate a diff that is similar to those used to generate patches. As this is not html, the css would have to be tweaked.
Og, could you please: - join a screenshot example of the resulting diff page with your version (I will also join one) - explain why you think that a classical diff view would be superior to a side-by-side diff view. It's possible that both views address different use cases.
Created attachment 126710 [details] Side by side diff view of two versions of a po file
Hi Claude, as I don't have vertimus operational on my checkout version of d-l, I do not have a html-rendered output of what my patch produces. However, this what you could expect as the output, minus html/css styles: +#: ../app/create_object.c:71 +#, c-format +msgid "'%s' creation failed" +msgstr "Falha ao criar \"%s\"" #: ../data/gnome-packagekit.schemas.in.h:49 -#, fuzzy msgid "Use WiFi (wireless LAN) connections to check for updates" -msgstr "Com que freqüência deve-se verificar por atualizações" +msgstr "Usa conexões WiFi (LAN sem fio) para verificar atualizações" It could be just the developer in me, but I find this to be relatively easier to follow than to look at side-by-side diffs. I don't know what use cases you have for this diff view and may well be that translators will cringe at looking at such output as the one I've suggested, but for someone doing the actual commit, a patch-like view may be easier to follow? Just my 2 cents. Cheers, Og
*** Bug 568432 has been marked as a duplicate of this bug. ***
Move this to the right component if you want to avoid duplicates ;-)