windows: Add testnet link to installer #8285

pull laanwj wants to merge 2 commits into bitcoin:master from laanwj:2016_06_testnet_link_windows changing 4 files +4 −0
  1. laanwj commented at 11:40 am on June 29, 2016: member

    Implements #8251

    testnet @jonasschnelli I’m not sure what tooling you used in 4c23743, but for some reason icotool, even though I’ve used the same sizes/bit depths as you, manages to produce a huge icon file:

    0  57964 Feb 18  2015 bitcoin.ico
    1 295606 Jun 29 13:01 bitcoin_testnet.ico
    
     0$ icotool -l bitcoin.ico > /tmp/bitcoin.lst
     1bitcoin.ico: clr_important field in bitmap should be zero
     2bitcoin.ico: clr_important field in bitmap should be zero
     3bitcoin.ico: clr_important field in bitmap should be zero
     4$ icotool -l bitcoin_testnet.ico > /tmp/bitcoin_testnet.lst
     5$ diff -du /tmp/bitcoin.lst /tmp/bitcoin_testnet.lst
     6[no output]
     7$ cat /tmp/bitcoin.lst
     8--icon --index=1 --width=48 --height=48 --bit-depth=4 --palette-size=16
     9--icon --index=2 --width=32 --height=32 --bit-depth=4 --palette-size=16
    10--icon --index=3 --width=16 --height=16 --bit-depth=4 --palette-size=16
    11--icon --index=4 --width=48 --height=48 --bit-depth=8 --palette-size=256
    12--icon --index=5 --width=32 --height=32 --bit-depth=8 --palette-size=256
    13--icon --index=6 --width=16 --height=16 --bit-depth=8 --palette-size=256
    14--icon --index=7 --width=256 --height=256 --bit-depth=32 --palette-size=0
    15--icon --index=8 --width=48 --height=48 --bit-depth=32 --palette-size=0
    16--icon --index=9 --width=32 --height=32 --bit-depth=32 --palette-size=0
    17--icon --index=10 --width=16 --height=16 --bit-depth=32 --palette-size=0
    

    Comparing binary, it looks like it’s not using PNG compression inside the icon.

    It also looks like it’s picking a smaller and lower-depth icon than it should.

  2. windows: Add testnet link to installer 0ce8e99ec8
  3. laanwj added the label Windows on Jun 29, 2016
  4. laanwj added the label Build system on Jun 29, 2016
  5. jonasschnelli commented at 12:16 pm on June 29, 2016: contributor

    utACK. An optimized version of the testnet icon can be found here: https://github.com/jonasschnelli/bitcoin/commit/6473d7305389f714eabdf30bd9c5ac176b8cd137

    I’m using the OSX “Icon Slate” app.

  6. laanwj commented at 1:13 pm on June 29, 2016: member

    Gah, seems I did something wrong while changing the color: testnet2

    What I did was take the 256x256 image and re-create all the other levels from that. But it looks like the smaller levels have relatively different amounts of whitespace around them? at least that would explain the differently-sized icon.

  7. jonasschnelli commented at 1:58 pm on June 29, 2016: contributor

    @laanwj: hmm.. I just re-created the testnet icon with the same properties like the mainnet icon: https://github.com/jonasschnelli/bitcoin/commit/d188c85e5ced31da39f76ed7aacad582d02cae72

    Maybe this one is more in align.

  8. MarcoFalke added this to the milestone 0.13.0 on Jun 29, 2016
  9. laanwj commented at 3:48 pm on June 29, 2016: member

    Unfortunately that one has a white block around it, and seems even smaller :(

    testnet3

  10. laanwj force-pushed on Jun 29, 2016
  11. laanwj commented at 4:38 pm on June 29, 2016: member

    Pushed an icon that has the right properties and size - rotated the hue with 70 degrees for every level of the icon individually.

    testnet4

    Could use file size optimization though :)

  12. jonasschnelli commented at 12:52 pm on June 30, 2016: contributor
    Tested ACK (on Windows 10).
  13. sipa commented at 1:12 pm on June 30, 2016: member

    Concept ACK, and the screenshots look good.

    However, I don’t understand how the second .lnk file is associated with the new icon.

  14. laanwj commented at 1:48 pm on June 30, 2016: member

    Tested ACK (on Windows 10).

    The new icon still needs optimization, would you be willing to do this?

    However, I don’t understand how the second .lnk file is associated with the new icon.

    It’s told to grab icon index 1 from the exe (index 0 is the default icon):

    0    CreateShortcut "$SMPROGRAMS\$StartMenuGroup\@PACKAGE_NAME@ (testnet, [@WINDOWS](/bitcoin-bitcoin/contributor/windows/)_BITS@-bit).lnk" "$INSTDIR\@BITCOIN_GUI_NAME@@EXEEXT@" "-testnet" "$INSTDIR\@BITCOIN_GUI_NAME@@EXEEXT@" 1
    

    This icon is added in the resource:

    0IDI_ICON2 ICON DISCARDABLE "icons/bitcoin_testnet.ico"
    
  15. jonasschnelli commented at 1:51 pm on June 30, 2016: contributor

    The new icon still needs optimization, would you be willing to do this?

    Sure. I guess this one should be fine: https://bitcoin.jonasschnelli.ch/bitcoin_testnet.ico

  16. windows: Add testnet icon for testnet link
    Overhauled testnet icon by Jonas Schnelli
    975a41dcc2
  17. laanwj force-pushed on Jun 30, 2016
  18. laanwj commented at 4:54 pm on June 30, 2016: member

    Sure. I guess this one should be fine:

    Added. Tested ACK, looks good now, and both icons are ~60kB.

  19. laanwj merged this on Jul 1, 2016
  20. laanwj closed this on Jul 1, 2016

  21. laanwj referenced this in commit da50997a3e on Jul 1, 2016
  22. codablock referenced this in commit 950116d50a on Sep 19, 2017
  23. codablock referenced this in commit a98af2bc64 on Dec 27, 2017
  24. codablock referenced this in commit 2c7ff7bf1d on Dec 27, 2017
  25. codablock referenced this in commit dcf36df675 on Dec 28, 2017
  26. codablock referenced this in commit f8a7a2c88d on Dec 28, 2017
  27. andvgal referenced this in commit 18807bc8e3 on Jan 6, 2019
  28. MarcoFalke locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-21 18:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me