#60 Manual: Use PNG images for all screenshots

closed-fixed
nobody
Docs (5)
3
2008-06-10
2008-03-19
Kimmo Varis
No

Currently there are many GIF screenshots in manual. We should unify them all to use one format. And PNG is a good one.

Discussion

  • Kimmo Varis
    Kimmo Varis
    2008-05-27

    Logged In: YES
    user_id=631874
    Originator: YES

    As of WinMerge 2.8.0 there are two gif screenshots left:
    - OpenDlg1.gif
    - sourcesafe1.gif

     
  • denisb
    denisb
    2008-06-06

    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

     
  • denisb
    denisb
    2008-06-06

    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.

     
  • Kimmo Varis
    Kimmo Varis
    2008-06-09

    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.

     
  • denisb
    denisb
    2008-06-10

    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

     
  • Kimmo Varis
    Kimmo Varis
    2008-06-10

    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.

     
  • Kimmo Varis
    Kimmo Varis
    2008-06-10

    • status: open --> closed-fixed