Sorry, please disregard the preceding proposed changes - I have a patch for just the two images mentioned by kimmov (and the two xml files that reference them), will submit if you like.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
When you submit a patch for some tracker item, please add a link to the patch in original item too. Makes it a lot easier to track what patch is for what item etc...
About the PNG images - I think we care only about manual files in this item. Good point about those other gif files there are. They are icon/status images in GUI and could be PNG files too in manual. Their original files are in Src/res. It would be nice to automatically convert those files (we need in manual) in Src/res to PNG. That way keeping GUI and manual in synch would be easier.
I don't mind converting web page images to PNG too. But there is no much point to convert for existing releases.
And if I remember correctly Tim usually takes screenshots to web pages from manual.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Logged In: YES
user_id=631874
Originator: YES
As of WinMerge 2.8.0 there are two gif screenshots left:
- OpenDlg1.gif
- sourcesafe1.gif
Logged In: YES
user_id=431345
Originator: NO
I propose to submit a patch after converting gifs to png, also changing references to them in files as summarized below. Okay with you?
Change image to png Change reference to png in
------------------- --------------------------
Docs\Users\Manual\images\up.gif Docs\Users\Manual\Compare_dirs.xml
Docs\Users\Manual\images\warning.gif Docs\Users\Manual\Manual-html.dsl
Docs\Users\Manual\screenshots\aborted.gif Docs\Users\Manual\Compare_dirs.xml
Docs\Users\Manual\screenshots\binary.gif Docs\Users\Manual\Compare_dirs.xml
Docs\Users\Manual\screenshots\binarydiff.gif Docs\Users\Manual\Compare_dirs.xml
Docs\Users\Manual\screenshots\equal.gif Docs\Users\Manual\Compare_dirs.xml (two places)
Docs\Users\Manual\screenshots\error.gif Docs\Users\Manual\Compare_dirs.xml
Docs\Users\Manual\screenshots\fileskip.gif Docs\Users\Manual\Compare_dirs.xml
Docs\Users\Manual\screenshots\folderskip.gif Docs\Users\Manual\Compare_dirs.xml
Docs\Users\Manual\screenshots\folderup.gif Docs\Users\Manual\Compare_dirs.xml
Docs\Users\Manual\screenshots\folderup_disable.gif Docs\Users\Manual\Compare_dirs.xml
Docs\Users\Manual\screenshots\lfile.gif Docs\Users\Manual\Compare_dirs.xml
Docs\Users\Manual\screenshots\lfolder.gif Docs\Users\Manual\Compare_dirs.xml
Docs\Users\Manual\screenshots\notequal.gif Docs\Users\Manual\Compare_dirs.xml
Docs\Users\Manual\screenshots\OpenDlg1.gif Docs\Users\Manual\Quick_start.xml
Docs\Users\Manual\screenshots\rfile.gif Docs\Users\Manual\Compare_dirs.xml
Docs\Users\Manual\screenshots\rfolder.gif Docs\Users\Manual\Compare_dirs.xml
Docs\Users\Manual\screenshots\sourcesafe1.gif Docs\Users\Manual\Version_control.xml
Externals\cppunit\src\msvc6\testrunner\ DynamicWindow\doc\cdxCDynamicWnd-DOC-big.gif Externals\cppunit\src\msvc6\testrunner\ DynamicWindow\doc\cdxCDynamicWnd-DOC.html
Externals\cppunit\src\msvc6\testrunner\ DynamicWindow\doc\cdxCDynamicWnd-DOC-small.gif Externals\cppunit\src\msvc6\testrunner\ DynamicWindow\doc\cdxCDynamicWnd-DOC.html
Web\1.7\images\icon.gif Web\WinMergePAD.xml
Web\.htaccess
Web\2.0\images\icon.gif Web\WinMergePAD.xml
Web\.htaccess
Web\2.2\images\dir-compare-status.gif Web\2.2\screenshots.php
Web\2.4\screenshots.php
Web\2.2\images\dir-compare.gif Web\2.2\screenshots.php
Web\2.4\screenshots.php
Web\2.2\images\file-compare.gif Web\2.2\screenshots.php
Web\2.4\screenshots.php
Web\2.2\images\icon.gif Web\WinMergePAD.xml
Web\.htaccess
Web\2.2\images\line-diff.gif Web\2.2\screenshots.php
Web\2.4\screenshots.php
Web\2.2\images\open-dialog.gif Web\2.2\screenshots.php
Web\2.4\screenshots.php
Web\2.6\screenshots.php
Web\2.8\screenshots.php
Web\2.2\images\screenshot.gif Web\.htaccess
winmerge\Web\2.2\index.php
winmerge\Web\2.4\index.php
Web\2.4\images\dir-compare-status.gif Web\2.2\screenshots.php
Web\2.4\screenshots.php
Web\2.4\images\dir-compare.gif Web\2.2\screenshots.php
Web\2.4\screenshots.php
Web\2.4\images\file-compare.gif Web\2.2\screenshots.php
Web\2.4\screenshots.php
Web\2.4\images\icon.gif Web\WinMergePAD.xml
Web\.htaccess
Web\2.4\images\line-diff.gif Web\2.2\screenshots.php
Web\2.4\screenshots.php
Web\2.4\images\open-dialog.gif Web\2.2\screenshots.php
Web\2.4\screenshots.php
Web\2.6\screenshots.php
Web\2.8\screenshots.php
Web\2.4\images\screenshot.gif Web\.htaccess
Web\2.2\index.php
Web\2.4\index.php
Web\2.4\images\statusbar.gif Web\2.4\limitations.php
Web\2.6\images\icon.gif Web\WinMergePAD.xml
Web\.htaccess
Web\2.6\images\open-dialog.gif Web\2.2\screenshots.php
Web\2.4\screenshots.php
Web\2.6\screenshots.php
Web\2.8\screenshots.php
Web\2.8\images\icon.gif Web\WinMergePAD.xml
Web\.htaccess
Web\2.8\images\open-dialog.gif Web\2.2\screenshots.php
Web\2.4\screenshots.php
Web\2.6\screenshots.php
Web\2.8\screenshots.php
Web\images\head.gif Web\css\screen.css
Web\images\icon.gif Web\WinMergePAD.xml
Web\.htaccess
Web\images\logo.gif Externals\expat\doc\reference.html
Web\images\padbutton.gif Web\page.inc
Don't change the following figures: they should be changed in build using DocBook stylesheet parameters.
Image Referenced in
----- -------------
Docs\Users\Manual\images\caution.gif Docs\Users\Manual\Manual-html.dsl
Docs\Users\Manual\images\home.gif none
Docs\Users\Manual\images\important.gif Docs\Users\Manual\Manual-html.dsl
Docs\Users\Manual\images\next.gif none
Docs\Users\Manual\images\note.gif Docs\Users\Manual\Manual-html.dsl
Docs\Users\Manual\images\prev.gif none
Docs\Users\Manual\images\tip.gif Docs\Users\Manual\Manual-html.dsl
Logged In: YES
user_id=431345
Originator: NO
Sorry, please disregard the preceding proposed changes - I have a patch for just the two images mentioned by kimmov (and the two xml files that reference them), will submit if you like.
Logged In: YES
user_id=631874
Originator: YES
When you submit a patch for some tracker item, please add a link to the patch in original item too. Makes it a lot easier to track what patch is for what item etc...
Patch item for this is:
#1988845 Manual: Use PNG images for all screenshots
http://winmerge.org/patch/1988845
About the PNG images - I think we care only about manual files in this item. Good point about those other gif files there are. They are icon/status images in GUI and could be PNG files too in manual. Their original files are in Src/res. It would be nice to automatically convert those files (we need in manual) in Src/res to PNG. That way keeping GUI and manual in synch would be easier.
I don't mind converting web page images to PNG too. But there is no much point to convert for existing releases.
And if I remember correctly Tim usually takes screenshots to web pages from manual.
Logged In: YES
user_id=431345
Originator: NO
Okay, thanks for the newbie tips. I've converted the additional images as suggested, submitted new patch:
#1989744 Use PNG images for all screenshots
http://winmerge.org/patch/1989744
Logged In: YES
user_id=631874
Originator: YES
I'm closing this item now as the the first patch is now in SVN trunk and 2.8 branch.