Grey

Results 122 comments of Grey

It seems to be related to the size of the web page. Shrinking the horizontal size of the web page allows the goal marker to show up just fine. Maybe...

After looking over the existing PR and making a vain attempt to merge `main` into this branch, I think I'd like to start from a fresh slate and make a...

Oh, I didn't notice #137 before opening the PR! I'll take a look at that and see if there's a way to achieve what I'm hoping for in there. >...

It looks like it would be pretty easy to migrate my changes onto your big refactor by changing [these fields](https://github.com/aevyrie/bevy_mod_picking/pull/137/files#diff-13a546f2a34c4d3f2eedd72db35b66e0cd40069bf2672a013c56e301fa3ef95eR103-R112) to optional and then tweaking the logic in [`update_highlight_asset`](https://github.com/aevyrie/bevy_mod_picking/pull/137/files#diff-13a546f2a34c4d3f2eedd72db35b66e0cd40069bf2672a013c56e301fa3ef95eR179). Does...

Maybe I can add an example that makes use of hover being disabled. At the very least you may want to try out the existing examples and see how you...

I suggest closing this PR in favor of #358 which combines this one with #239 and resolves the conflicts between them.

Your points are well-taken. I was curious about investigating the significance of the performance difference, so I created a [simple performance](https://github.com/mxgrey/universal_robot_test) test for the function. I've created a branch that...

Thanks for inspecting the test. I've made the changes you suggested (the repo has also been updated), and I've seen the same trend that you're reporting. When `limited` is `false`...

If @a-price is willing to merge the PR a-price#1, then I'll be happy to approve.

> Preserving the ABI: Since the binary interface to the library from outside the package is, by my understanding, currently using undefined behavior, I don't have a huge problem with...