Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add overlay support to the model #77

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Add overlay support to the model #77

wants to merge 2 commits into from

Conversation

Gudahtt
Copy link
Member

@Gudahtt Gudahtt commented Oct 18, 2021

Each chunk can now define an overlay. An overlay is drawn over the base model, and can have its own color, gradient, and mask properties. An overlay can be used to apply a pattern over the entire model at once.

Using an overlay does have a performance impact because each polygon in a chunk with an overlay gets duplicated. But it makes it dramatically easier to apply a pattern over the entire model at once.

A demo of this functionality has been added, called "overlay". It is not the nicest looking example, but it does demonstrate the functionality well.

@Gudahtt
Copy link
Member Author

Gudahtt commented Oct 18, 2021

This depends upon #76

@Gudahtt Gudahtt mentioned this pull request Oct 18, 2021
Base automatically changed from add-mask to main October 18, 2021 19:38
@Gudahtt Gudahtt marked this pull request as ready for review October 18, 2021 19:39
@Gudahtt Gudahtt requested a review from a team as a code owner October 18, 2021 19:39
Each chunk can now define an overlay. An overlay is drawn over the base
model, and can have its own `color`, `gradient`, and `mask` properties.
An overlay can be used to apply a pattern over the entire model at
once.

Using an overlay does have a performance impact because each polygon in
a chunk with an overlay gets duplicated. But it makes it dramatically
easier to apply a pattern over the entire model at once.

A demo of this functionality has been added, called "overlay". It is
not the nicest looking example, but it does demonstrate the
functionality well.
@Gudahtt Gudahtt marked this pull request as draft February 6, 2025 17:36
@Gudahtt
Copy link
Member Author

Gudahtt commented Feb 6, 2025

I don't remember why I wrote this 🤔

Needs a few updates now, so I've moved it to draft.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant