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

Modify zoom of bicycle, theatre, water_tower #1627

Closed
nvkelso opened this issue Sep 24, 2018 · 2 comments
Closed

Modify zoom of bicycle, theatre, water_tower #1627

nvkelso opened this issue Sep 24, 2018 · 2 comments

Comments

@nvkelso
Copy link
Member

nvkelso commented Sep 24, 2018

tl;dr

Only bicycle, theatre, and water_tower are relevant here. See comment below.

            hide-until-z17-no-area:
                filter:
                    area: false
                    $zoom: { max: 17 }
                    # note: atm and drinking_water appear hear and in area versions
                    kind: [accountant, administrative, advertising_agency, animal, architect, association, atm, bakery, bed_and_breakfast, bicycle, bicycle_parking, bicycle_rental_station, books, butcher, car, car_repair, chalet, clothes, company, computer, consulting, convenience, doityourself, drinking_water, dry_cleaning, educational_institution, emergency_phone, employment_agency, estate_agent, fashion, financial, florist, foundation, gift, government, greengrocer, guest_house, hairdresser, hostel, hotel, insurance, it, jewelry, lawyer, mast, memorial, mobile_phone, motel, newspaper, ngo, notary, optician, parking, pet, physician, playground, political_party, post_box, religion, research, subway_entrance, tax_advisor, telecommunication, telephone, theatre, therapist, toilets, traffic_signals, travel_agent, water_tower]
                draw:
                    mapzen_icon_library:
                        visible: false

@nvkelso nvkelso added this to the v1.6.0 milestone Sep 25, 2018
@nvkelso
Copy link
Member Author

nvkelso commented Dec 12, 2018

Change these, for no-area (node) case:

Maybe:

  - filter: {shop: bicycle}
    min_zoom: { clamp: { min: 15, max: 17, value: { sum: [ { col: zoom }, 2.5 ] } } }
    output:
      <<: *output_properties
      kind: bicycle

Maybe:

  - filter:
      amenity: theatre
    min_zoom: { clamp: { min: 15, max: 17, value: { sum: [ { col: zoom }, 2.7 ] } } }
    output:
      <<: *output_properties
      kind: theatre

Suggest riffing on what we do for obelisk for water_tower, maybe:

  - filter: {man_made: water_tower}
    min_zoom:
      lookup:
        key: { call: { func: mz_to_float_meters, args: [ { col: height } ] } }
        op: '>='
        table:
          - [ 15, 20 ]  # z15 if height >= 20m
          - [ 16, 10 ]  # z15 if height >= 10m
        default: 17

These are already zoom 17, no change needed:

accountant
administrative
advertising_agency
animal
architect
association
atm
bakery
bed_and_breakfast
bicycle_parking
bicycle_rental_station
books
butcher
car
car_repair
chalet
clothes
company
computer
consulting
convenience
drinking_water
dry_cleaning
educational_institution
emergency_phone
employment_agency
estate_agent
fashion
financial
florist
foundation
gift
government
greengrocer
guest_house
hairdresser
hostel
hotel
insurance
it
jewelry
lawyer
mast
memorial (already lots of complicated logic added for this one)
mobile_phone
motel
newspaper
ngo
notary
optician
parking
pet
physician
playground
political_party
post_box
religion
research
subway_entrance
tax_advisor
telecommunication
telephone
therapist
toilets (already 18)
traffic_signals
travel_agent
doityourself

@nvkelso nvkelso changed the title Hide early office related when no area Modify zoom of bicycle, theatre, water_tower Dec 12, 2018
@nvkelso nvkelso modified the milestones: v1.6.0, v1.6.1 Dec 14, 2018
@ghost ghost assigned zerebubuth Dec 28, 2018
@ghost ghost added the in review label Dec 28, 2018
@nvkelso
Copy link
Member Author

nvkelso commented Jan 23, 2019

QA verified.

@nvkelso nvkelso closed this as completed Jan 23, 2019
@ghost ghost removed the in review label Jan 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants