new icons for windows, linux, etc. #2477

pull jonasschnelli wants to merge 3 commits into bitcoin:master from jonasschnelli:new_icons changing 7 files +67 −177
  1. jonasschnelli commented at 6:42 PM on April 6, 2013: contributor

    According to pull 2444 (https://github.com/bitcoin/bitcoin/pull/2444):

    • new windows and linux icons.
    • doxygen-icon should also be fine.

    The windows icon contains multiple resolutions up to 256px. Is someone capable of doing a Vista and W8 test? Who takes care of the linux test?

  2. moved bitcoin.svg to correct folder (qt/res/src)
    Signed-off-by: Jonas Schnelli <jonas.schnelli@include7.ch>
    5675ae2f11
  3. new windows and linux icons
    - new windows .ico contains multiple resolutions up to 256px
    - new testnet (green) icon
    - new png icon for llinux, etc.
    - new doxygen icon
    - changed the assets-attribution.txt
    
    Signed-off-by: Jonas Schnelli <jonas.schnelli@include7.ch>
    27cc0d7fd2
  4. BitcoinPullTester commented at 10:19 PM on April 6, 2013: none

    Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/27cc0d7fd248fac60705609c51ce25e4db04a758 for binaries and test log. This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/ Contact BlueMatt on freenode if something looks broken.

  5. laanwj commented at 7:57 AM on April 7, 2013: member

    Cute!

  6. in doc/assets-attribution.txt:None in 27cc0d7fd2 outdated
       0 | @@ -1,5 +1,8 @@
       1 |  Icon: src/qt/res/icons/clock*.png, src/qt/res/icons/tx*.png,
       2 | -      src/qt/res/src/*.svg
       3 | +      src/qt/res/src/clock_green.svg, src/qt/res/src/clock1.svg
       4 | +	  src/qt/res/src/clock2.svg, src/qt/res/src/clock3.svg
    


    Diapolo commented at 10:51 AM on April 7, 2013:

    Can you indent these and the 2 below with the above line :)?


    jonasschnelli commented at 7:10 AM on April 8, 2013:

    Because the old text "src/qt/res/src/*.svg" had covered the newly generated src/qt/res/src/bitcoin.svg i had to "rewrite it arithmetically". :) I think it's okay like this.

  7. in doc/assets-attribution.txt:None in 27cc0d7fd2 outdated
      37 | @@ -35,7 +38,7 @@ Designer: http://www.everaldo.com
      38 |  Icon Pack: Crystal SVG
      39 |  License: LGPL
      40 |  
      41 | -Icon: src/qt/res/icons/bitcoin.png, src/qt/res/icons/toolbar.png
      42 | +Icon: src/qt/res/icons/toolbar.png
    


    Diapolo commented at 10:55 AM on April 7, 2013:

    Who did toolbar_testnet.png btw. and do we need updated version of these also?


    laanwj commented at 5:37 AM on April 8, 2013:

    I did the toolbar_testnet icon. The toolbar icon is scaled so far that this change doesn't matter to it (and for example, it doesn't have the drop shadow in the first place).

  8. Diapolo commented at 10:58 AM on April 7, 2013: none

    I think the bitcoin_logo_doxygen.png looks a little blurry compared to the former version. I'm not yet sure if I like the new testnet color, I will wait for pull-tester to build and then try it out :). @jonasschnelli Very nice work, I have no passion to be a graphics-artist and love that work you show use here.

  9. laanwj commented at 5:40 AM on April 8, 2013: member

    I also liked the darker testnet color a bit better. Having said that, it's only the testnet, the poison green is meant to be kind of a warning color :) In Andreas Schildbach's bitcoin-wallet client a different take on the warning pattern is used: https://play.google.com/store/apps/details?id=de.schildbach.wallet_test&hl=en

    ACK, we should certainly merge this for 0.8.2

  10. Diapolo commented at 6:05 AM on April 8, 2013: none

    I'm fine with testnet, if you are... ACK for the mainnet icon ;).

  11. jonasschnelli commented at 7:13 AM on April 8, 2013: contributor

    In my eye, the testnet color should just be "visible" somehow. Because mostly devs are using the testnet icon, it has – in my eyes – not the same quality requirements. But nevertheless, the quality is good. :)

  12. new icons: overhaul oxygen icon (was to blurry)
    Signed-off-by: Jonas Schnelli <jonas.schnelli@include7.ch>
    a653191f74
  13. jonasschnelli commented at 8:38 AM on April 8, 2013: contributor

    @Diapolo: right, the doxygen icon was blurry. It's fixed and pushed now.

  14. BitcoinPullTester commented at 9:12 AM on April 8, 2013: none

    Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/a653191f748f56addf0cfa219319cfc5358b962d for binaries and test log. This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/ Contact BlueMatt on freenode if something looks broken.

  15. laanwj referenced this in commit 70c71a6a2a on Apr 8, 2013
  16. laanwj merged this on Apr 8, 2013
  17. laanwj closed this on Apr 8, 2013

  18. Diapolo commented at 7:31 AM on April 9, 2013: none

    @jonasschnelli I'm now using your current icons with my local build and I think you should also update the toolbar.png and toolbar_testnet.png versions. Specially on testnet the combination of the new color with the old one in the Windows system-tray looks really weird/ugly :). Also I still think the old color was a little more friedly, the new one doesn't fit into the color scheme (at least on Windows).

    testnet

  19. jonasschnelli commented at 7:45 AM on April 9, 2013: contributor

    @Diapolo your right. I did not take care of the testnet toolbar. I will no update the color more according to the old testnet color (was more warm) and also take care of the testnet toolbar.

  20. Diapolo commented at 11:00 AM on April 10, 2013: none

    @jonasschnelli Can you take a look? I think the new icon is missing a little transparency or fading on the outer ring and seems smaller (Or is this intended?) for me on Windows.

    (left old / right new) Icon

  21. jonasschnelli commented at 12:00 PM on April 10, 2013: contributor

    @Diapolo let me startup my Windows WM...

  22. jonasschnelli commented at 12:44 PM on April 10, 2013: contributor

    fixed and placed in PR #2497

    preview: preview

    ... and the remove of the transparency was a conscious decision.

  23. laudney referenced this in commit 48fdd08e6b on Mar 19, 2014
  24. DrahtBot 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: 2026-04-24 12:16 UTC

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