gui: New filesave icon, updated copyright #16584

pull GChuf wants to merge 1 commits into bitcoin:master from GChuf:branch4 changing 2 files +15 −6
  1. GChuf commented at 8:45 PM on August 11, 2019: contributor

    Updated the filesave icon (the original was the same as the receive.png) Updated and fixed some typos in the copyright file

  2. GChuf commented at 8:48 PM on August 11, 2019: contributor

    Sorry for the initial mess everyone. I created a cleaner pull request with only 1 commit. I still don't have any screenshots, else I would upload them.

  3. emilengler commented at 9:02 PM on August 11, 2019: contributor

    See #16583

  4. [QT] New filesave icon 08b18f5cb8
  5. DrahtBot added the label GUI on Aug 11, 2019
  6. DrahtBot added the label Scripts and tools on Aug 11, 2019
  7. fanquake removed the label Scripts and tools on Aug 11, 2019
  8. fanquake renamed this:
    [QT] New filesave icon, updated copyright
    gui: New filesave icon, updated copyright
    on Aug 11, 2019
  9. fanquake commented at 12:07 AM on August 12, 2019: member

    Concept ACK on an icon that is more obviously a "save" action, and different from receive. However I don't see these icons on macOS, so will leave it up to others to comment. In any case, the new icon file can be compressed by about ~50%.

    Old filesave icon: filesave_old

    New filesave icon: filesave_new

  10. jonasschnelli commented at 8:03 AM on August 12, 2019: contributor

    I'm not a license guy... but is the Creative Commons BY 3.0 compatible with our MIT license? @luke-jr @wtogami maybe?

  11. GChuf commented at 9:03 AM on August 12, 2019: contributor

    @fanquake what do you mean by compression - reducing image size in pixels, indexing png, something else? I thought Qt takes care of compression when compiling.

    As for licences compatibility, I found this and this. I think for this case it should be fine.

  12. emilengler commented at 11:47 AM on August 12, 2019: contributor

    @jonasschnelli Yes but I think only when we add the name of the author to our COPYING file like

    Copyright (c) 2009-2019 The Bitcoin Core developers
    Copyright (c) 2009-2019 Bitcoin Developers
    Copyright (c) 2019 (or whatever date he made the icon) THE_GUY_WHO_CREATED_THIS_ICON
    

    Also I'm not a lawyer but CC-BY means in general that we only need to mention the author, probably in the copying file. Although it might be possible if we create a THANKS file which includes names like the guy who made the icon. Many other projects have this

  13. MarcoFalke commented at 12:25 PM on August 12, 2019: member

    Yeah, I think we'd have to update the GUI About dialog to include the authors of the icons.

  14. jonasschnelli commented at 1:22 PM on August 12, 2019: contributor

    Yeah, I think we'd have to update the GUI About dialog to include the authors of the icons.

    I think it's not worth the effort for a single icon. Maybe @gapeman can take or modify an existing icon from the typicons family?

    Also, we should not forget that the floppy-disk icon may appeal to "older guys" (like us?)... but those disks are out of use since years and have no longer anything to do with "persistence store". It's like showing a phone-hook-modem when meaning "the internet".

  15. MarcoFalke commented at 1:31 PM on August 12, 2019: member

    @jonasschnelli We don't mention Stephen Hutching in the about dialog right now. Even if that is not required by the license, it feels like a nice gesture to do.

    Agree that we shouldn't use a floppy disk as the "save icon". Apart from ms word no one is using that anymore these days.

  16. jonasschnelli commented at 1:58 PM on August 12, 2019: contributor

    @jonasschnelli We don't mention Stephen Hutching in the about dialog right now:

    See #6367 (comment). Stephen agreed to distribute it under MIT. I think either we mention all gfx designers in the about dialog or none. I guess people accept and agree that this project is not about persona.

  17. GChuf commented at 4:41 PM on August 12, 2019: contributor

    I agree that it's not worth the effort for a single icon. None of the typicons seem to really represent saving, so I was thinking about creating an icon myself. I also came across these icons, all licensed under MIT, which might come in handy in the future: [1] [2]

    And even though floppy disks aren't used anymore, we all know what the icon means and I think it's still being used extensively. Notepad++ comes to mind and my search results across various search engines all output floppy disk icons when searching for a "save icon". I'm not aware of any other "standard" save icons and personally would like to see this trend survive.

  18. MarcoFalke commented at 5:27 PM on August 12, 2019: member

    The only icon I found was the one in libreoffice, which looks similar to our old save icon.

    Screenshot from 2019-08-12 13-26-06

  19. GChuf commented at 7:09 PM on August 12, 2019: contributor

    I'm not sure what the point was, @MarcoFalke. Version 5.1.6.2 of libreoffice that I have installed uses floppy disk icons, but I don't want to get into a conversation which programs use what.

    Here's libreoffice floppy disk icon screenshot (which I like) and should probably be hidden somewhere in the official github libreoffice repo ... Screenshot_52

    I think we should all just vote for either floppy disk or something else.

  20. emilengler commented at 7:20 PM on August 12, 2019: contributor

    I still think everyone knows what is meant by a floppy disk :)

  21. GChuf commented at 8:23 PM on August 13, 2019: contributor

    I've made an icon myself from scratch: filesave crushed

    I've also found the optimize-pngs.py script fanquake was probably referring to.

    The result in bitcoin-qt (I spent a while optimizing the icon so it looks great in qt): Screenshot_59

    I'd like to put the file under public domain. What do you think?

  22. kristapsk commented at 8:43 PM on August 13, 2019: contributor

    I agree that most people know what floppy disk icon means, but it will become less and less intuitive in the future and I'm also not sure is the right way to go. There are more and more computer users nowadays that haven't seen any floppy disk in their entire life.

  23. emilengler commented at 11:14 PM on August 13, 2019: contributor

    I personally think this icon doesn't fit in that good. I prefer the older one as it looks more like the others, but this is my opinion so NACK sorry :(

  24. laanwj commented at 10:40 AM on August 14, 2019: member

    If this requires any special license exemptions it's not worth it, IMO. We simplified the licensing a lot, one time, by switching from gathered icons to icons from one icon set. From a practical perspective that's great and also ensures consistency.

    No opinion on floppy disk versus box-with-arrow.

    However I don't see these icons on macOS, so will leave it up to others to comment.

    BTW: are menu icons even still a thing in modern GUI? At least GNOME hasn't displayed them anymore for a long time. Same for MacOSX apparently. Might consider removing them completely.

  25. MarcoFalke commented at 11:47 AM on August 14, 2019: member

    Just tested on Fedora 30, and it looks like they aren't even shown. So I tend to agree with @laanwj

    Screenshot from 2019-08-14 07-45-26

  26. jonasschnelli commented at 11:51 AM on August 14, 2019: contributor

    Closing this due not worth the effort Thanks anyways @gapeman for the intention to improve. Maybe this leads to a PR removing all the menu icons...

  27. jonasschnelli closed this on Aug 14, 2019

  28. promag commented at 10:15 PM on August 14, 2019: member

    Also agree with removing the icons.

  29. jonasschnelli referenced this in commit 93bacb8cc9 on Aug 16, 2019
  30. GChuf deleted the branch on Aug 16, 2019
  31. luke-jr commented at 6:45 PM on August 23, 2019: member

    Menu icons are still used in KDE at least.

  32. ShengguangXiao referenced this in commit dc50b78b81 on Aug 28, 2020
  33. Fabcien referenced this in commit c7066157da on Dec 24, 2020
  34. DrahtBot locked this on Dec 16, 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-13 21:14 UTC

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