-
Notifications
You must be signed in to change notification settings - Fork 831
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
move library icon to SVG #1228
move library icon to SVG #1228
Conversation
I don't like the horizontal space - now it looks more like an arrow down or a gift than a book. |
I can't follow you regarding the gift I have to say. |
Well, the lower part of the package with the gift. =} Anyway, I can't follow the horizontal space in the book icon - what does it represent? |
It is intended to represent the pages, the thickness of the book. |
Hm, I was not sure, since I expect it to be on top. And I guess the cover is too short (vertical cover line should be longer than horizontal - this one is too square). |
That seems to be a problem with this icon. The book is meant to show the inside but some are seeing the back cover (me too sometimes). I will try to adjust the fake perspective. |
Yes, tried it already but didn't work so well. |
I still consider it strange because of a square aspect ratio (if I understand correctly those small parts on the bottom are the thickness/z axis, so they don't count for height) - rectangles with one dimension longer are more common: https://commons.wikimedia.org/wiki/File:Comparison_paper_sizes.svg |
1eff1e1
to
27e7db7
Compare
I have updated the icon and removed the z-axis part. |
I think most names of the poi will clarify the icon for those people who don't recognize it. |
Unfortunately at least for me the new icon is not recognizable and current one is. |
I have experimented quite a bit with different options before the last update and I'm afraid I will not be able to come up with a different/better version. |
These are some variations: |
Both are much better for me! |
@vholten Are your previews output by TileMill or did you do some image editing? If TileMill, did you use PNG or SVG? I'm asking because the output looks far too crisp for me regarding all the diagonal features. |
@nebulon42 My previous previews were done by image editing (downsampling the icons to 16 pixel width). |
Looks a bit blurry at the edges, but not bad. Maybe it helps to have the parts at the left/right side in solid colour as it is done for the lower part? I suggest that you create another Pull Request for it. |
@vholten It looks really well, better than the current one. |
@vholten I like them too. |
OK, I will try some variations and create a new pull request. |
For what it's worth, tester recognizes the image proposed in this PR but prefers the current image. |
Thanks, but let's wait for @vholten's PR. |
I tried to recreate @vholten's proposal and it looks like this: @math1985: Here I tried to use transparency (0.1) for the fill. What do you think? As the outlines are a bit thin and the fill (pages) are quite big I think the icon is not prominent enough compared to the others, especially at lower zoom levels: |
Please give feedback on the new icon so we can move this forward, thanks. |
I'm very busy at the moment, but will get back to it as soon as I have time. |
I like this icon - it's a bit less visible then current one, but then I think libraries are now too visible. New icon is satisfying recognizable for me. |
I also like the new icon. It's more recognizable then the 'arrow', and as far as I'm concerned still prominent enough. We could also consider swapping your proposed library and book shop icon. A (messy) stack of books reminds me more of a library than a book shop. |
I also like the new icon. I think it's visible enough, especially when combined with the label. I'm not so sure about swapping the icon with book shop. In my experience, the open book is a fairly universal symbol for representing libraries. Regardless, the colour of the icon and label is different for the two, so it should be relatively easy to determine which is the amenity and which is the shop. |
Situation in Poland is the same - IMHO this symbol or similar is more likely to represent library. |
I have to delay this a bit because this involves transparency and I have to address https://github.com/nebulon42/osmic/issues/3 first to see if the transparency is still needed. We then will have to decide if we want to have icon halos in here too (#1189). |
@nebulon42 Is there anything blocking this PR? This icon is a nice improvement, is there anything that needs to be done before reviewing and hopefully merging it? |
@matkoniecz After careful consideration I cannot use the last proposed variant, because of the large inner holes that lead to difficulties with legibility on noisy background. I therefore have reverted the library icon in my repository and will go on proposing my version as depicted in the preview in the first post. You can still find the other icon here if you choose to go ahead with that version. |
@nebulon42 Yes, it is not working in cases like this example. |
Do the pages have to be completely translucent or fully brown, or there could be a shade? |
Technically a transparent fill (say 10/15/20%) could work. But translucent fills clash with halos, so I cannot add that. (Although halos are not considered in this repository.) |
Rather than transparency, is it possible to use a lighter, but still solid, shade? Or can it only have on/off for each pixel? I'm not completely familiar with how this all works, but I envision something like having the pages coloured by the primary colour -50% saturation, for example. |
@lest69 When using Please tell me if you consider to merge this, otherwise I will close it. |
I'm fine with merging the current proposal, what do the others think? |
For me it's just passable - visually not better than current icon (too strong), but not worse. |
I'd rather keep the current one or use something similar to what was proposed in the PR comments. I don't like the one in the PR itself. |
I'm closing this, because we are re-iterating the same discussion over and over again. |
Implements part of #1165 by moving the icon for
amenity=library
to SVG.Icon: nebulon42/osmic/library-16.svg
before

after
