Conversation
|
I see that it is a draft - what remains to be done? Do you need some help? |
|
Pretty much just testing, or giving feedback on the selected images. |
|
Can we also add support for artwork_type=street_art ? |
I think it could work, I think we would need to title it as "other street art". |
Yeah, "other street art" sounds good. Perhaps this picture is somewhat representative (can't really be put in other categories): |
I have the feeling this might be hard to recognized when scaled down for smaller screen sizes. |
Perhaps crop the prominent part of it? Other examples (also on that wiki) that I've had is likely even harder to recognize. On the other hand, most of street art matches on wikimedia commons are mostly E.g. maybe this one has better contrast? Or perhaps looking if something else already mapped in OSM as |
westnordost
left a comment
There was a problem hiding this comment.
It is a precedent to include the JPEG source graphics in the /res/graphics directory in this repository. We usually only do this for the (vector) graphics. There is another authors.txt in /app/src/androidMain/res/ which contains all the attribution plus an URL to the source. If there is no good reason to choose a different format here, I'd ask you to do that, for consistency.
app/src/commonMain/kotlin/de/westnordost/streetcomplete/quests/artwork/ArtworkType.kt
Outdated
Show resolved
Hide resolved
app/src/commonMain/kotlin/de/westnordost/streetcomplete/quests/artwork/ArtworkTypeItem.kt
Outdated
Show resolved
Hide resolved
app/src/commonMain/kotlin/de/westnordost/streetcomplete/quests/artwork/ArtworkTypeItem.kt
Show resolved
Hide resolved
…/artwork/ArtworkTypeItem.kt Co-authored-by: Tobias Zwick <newton@westnordost.de>
|
|
As for the icon, how about an icon like this: https://www.svgrepo.com/svg/359716/art-gallery ? |
|
Well, like this, I guess. That one is black and white of course. Or maybe take the icon of some open source (desktop?) paint app? It would be a nice gimmick. GIMP icon although arguably the biggest one is maybe not recognized well enough? Maybe MyPaint: https://github.com/mypaint/mypaint ? |
|
The other street art icon is very good, because it shows two different kinds of street art. |
|
This looks to be the current (flat) gimp icon: https://commons.wikimedia.org/wiki/File:The_GIMP_icon_-_v3.0.svg I am kind of intrigued.... |
|
It looks like this is done minus the quest icon. Good work! |
|
(By the way, tried out MyPaint for the first time. It looks like a simpler Krita, i.e. geared towards "real" painting (with an actual canvas pattern as background). Must be pretty fun when you have a pen tablet. But boy is it slow (on my computer).) Anyway, what do you think about trying the GIMP icon? At least among (FOSS) nerds, it should be well known and would be a nice gimmick. I expect there is a considerable intersection between OSM nerds and FOSS nerds. |
I believe that one might be (in a process of being) trademarked: https://uspto.report/TM/99400787 |
|
What does that mean? IANAL. In the linked filing, it says that it is a
|
|
TL;DR: it might, or might not, be problematic. Best to clarify with the GNOME foundation.
Trademark means that the usage of logo enjoys certain protections under the law. Anyone may use it freely1 to refer exactly to GIMP, but to use it to refer to different things (inside protected class) without having permission from trademark owner would be considered violation, AFAIK.
Yeah, that quote is why (instead of straight-out claiming StreetComplete may not use it) I merely suggested it would be best to verify with GNOME foundation if we can use it for that purpose. Because, its registration also says that its Primary International Class includes "surveying [apparatus and instruments]" and "data processing equipment and computers" which StreetComplete app on mobile phone might be matching. But, again, it might not, as the text you quoted might be taking precedence2 which is why I recommended seeking clarification/permission from trademark owner might be a good idea. Footnotes |
|
For reference: icon added in 4303311 |
|
Note the strings were added to the wrong strings.xml file. Right now, we have two identical |

Closes #2125
I copied most of the implementation from SCEE (Helium314#796) by @vfosnar, with some modifications:
I removed landart for the following reasons: