Skip to content
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

Add special rendering for shop=laundry and shop=dry_cleaning #1411

Closed
kocio-pl opened this issue Mar 18, 2015 · 5 comments · Fixed by #1595
Closed

Add special rendering for shop=laundry and shop=dry_cleaning #1411

kocio-pl opened this issue Mar 18, 2015 · 5 comments · Fixed by #1595

Comments

@kocio-pl
Copy link
Collaborator

We should start rendering shop=laundry and shop=dry_cleaning, probably with this icon from osmfr-cartocss fork:

https://github.com/cquest/osmfr-cartocss/blob/master/symbols/laundry.svg

[It was proposed in meta-issue #1402]

@kocio-pl kocio-pl changed the title Add rendering for shop=laundry and shop=dry_cleaning Add special rendering for shop=laundry and shop=dry_cleaning Mar 18, 2015
@matthijsmelissen matthijsmelissen added this to the New features milestone Apr 29, 2015
@kocio-pl
Copy link
Collaborator Author

My sketches:

  1. laundry-14-1
  2. laundry-14-2

@kocio-pl
Copy link
Collaborator Author

Some more:
3) laundry-14-3
4) laundry-14-4
5) laundry-14-5

I like 5) the most, then 3).

@kocio-pl
Copy link
Collaborator Author

Test - for me it works good:

laundry-z19

@lest69
Copy link

lest69 commented May 29, 2015

It looks too much like a camera to me.

What about a shirt partially inside of a washing basin? It may be old-fashioned and doesn't accurately depict the dry-cleaning process, but it would convey the concept of the cleaning of clothing. I'm not sure if you can fit that into the available space, though.

@kocio-pl
Copy link
Collaborator Author

The idea was not bad, but I think it doesn't work too good: laundry 16

(BTW: the merge of PR was done just before the moment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants