Skip to content
This repository has been archived by the owner on Oct 7, 2024. It is now read-only.

Adding example of "revealed" polygon hole with outline #1050

Merged
merged 2 commits into from
May 24, 2019

Conversation

langsmith
Copy link
Contributor

@langsmith langsmith commented May 7, 2019

This pr adds an example of an outlined polygon hole in a larger shaded area in San José (cc @1ec5). Inspired by
Screen Shot 2019-05-07 at 4 02 32 PM

ezgif com-resize (3)

@langsmith
Copy link
Contributor Author

Example like this could be used to create something like @ClareTrainor 's map 👇 😃

Screen Shot 2019-04-04 at 10 55 24 AM

@langsmith langsmith force-pushed the ls-shaded-polygon-hole-with-outline-example branch 2 times, most recently from 031016c to 9ce9bbd Compare May 15, 2019 22:14
@tobrun tobrun requested review from LukasPaczos and tobrun and removed request for osana May 21, 2019 08:23
Copy link
Member

@LukasPaczos LukasPaczos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks cool! A couple of comments below. Do you think it'd make sense to restrict the camera to the outer bounding box?

@langsmith langsmith force-pushed the ls-shaded-polygon-hole-with-outline-example branch from 9ce9bbd to c4100ef Compare May 23, 2019 19:24
@langsmith langsmith force-pushed the ls-shaded-polygon-hole-with-outline-example branch from c4100ef to 84504b9 Compare May 23, 2019 19:36
@langsmith
Copy link
Contributor Author

All feedback addressed, @LukasPaczos . Anything else?

@langsmith
Copy link
Contributor Author

Do you think it'd make sense to restrict the camera to the outer bounding box?

I also added camera bounds.

ezgif com-resize (3)

Copy link
Member

@LukasPaczos LukasPaczos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚢

@langsmith langsmith merged commit 02d7559 into master May 24, 2019
@langsmith langsmith deleted the ls-shaded-polygon-hole-with-outline-example branch May 24, 2019 12:20
@jealalbar
Copy link

jealalbar commented May 27, 2019 via email

@langsmith langsmith mentioned this pull request Jun 13, 2019
5 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants