-
Notifications
You must be signed in to change notification settings - Fork 12
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
Port dots styles to use texture #32
Comments
Porting to @bcamper What about |
Mmmm... I just notice that all those dots have white |
Next step making the texture! @bcamper I have two ways to go here:
Seams the first one is slightly more expensive but more reusable. Also I would love to use some blocks for this... are we ok for adding tangram-blocks?? cc @nvkelso @bcamper |
It might be worth trying both approaches to see what the performance/size difference is. Maybe start with a single dot. But I'd like all the dependencies for refill in the project itself -- it's been difficult for me to deal with all the imports in Tron (though it was a good experiment), and I'm not yet convinced we want to bring that complexity to other house styles, so I'd rather keep it as an embedded style in here for now. |
👍 |
In general, performance is more important for these house styles than flexibility. Since a second dot texture probably won't add much size, I would likely prefer that (let's check the size?). You can also make a generic version for Blocks, with a single texture, and keep an optimized version in this file. |
Cool... @sensescape could you confirmam that the |
@patriciogonzalezvivo ok! reviewing the dot sizes |
thanks! |
sooo... I tried with fixed size dot textures... and seams they don't scale right... so I'm taking the other approach: ONE SDF TO RULE THEM ALL! |
sorry about this mess, so many dots! looks like I'm using them all |
@sensescape THANKS! I hope you don't mind I simplify the names in this commit a1b463a some stiles where only call by just another style |
yes, please @patriciogonzalezvivo |
All changes are here #33 ready for review and merge : ) |
We can simplify and improve performance of the dots-based styles by making them sample from a dot texture (that can be further modified in the shader as needed).
The text was updated successfully, but these errors were encountered: