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

Add r3f doc for community components #3378

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

jo-chemla
Copy link

@jo-chemla jo-chemla commented Oct 11, 2024

Following the issue #3377 I opened earlier today, here is a PR that adds a page to the Getting Started section listing external/community r3f components that have not been merged to drei or part of the pmndrs github org. Preview is here!

I've added all the components I know of, feel free to add more before merging!

Not docs does not seem to take the screenshot_url param visible here of the <Codesandbox /> component.

Copy link

codesandbox-ci bot commented Oct 11, 2024

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit d32ca6a:

Sandbox Source
example Configuration

@abernier
Copy link
Member

as discussed on X, maybe a bit redundant with Discord's Showcase channel i think

@abernier
Copy link
Member

abernier commented Oct 13, 2024

or we could write a <DiscordShowcases staffPicked> component (using Discord api), that displays all "staff-picked" tagged showcases, but again it would be redundant with such view that Discord already implement https://discord.com/channels/740090768164651008/1047453248044412968

  1. => at least (without implementing anything into docs.) maybe we can introduce that staff-picked tag?
  2. => Or, instead of an arbitrary choice by the "staff", we could let reactions like 😍/🔥 speak for themselves. We can maybe just put a link to Discord Showcase into the docs. website into the examples page, nothing more

I'm for 2.

@jo-chemla
Copy link
Author

Thanks for the feedback on X and here! Here is my take regarding the points you mentioned:

I've been a long-time user of r3f/drei, but never saw a community collection of projects that did gather large threejs projects that expose react/r3f components. I'm thinking about large-scale projects rather than small set of utility reusable components, like other kind of renderers - 3d-tiles renderers from gkjohnson or nytimes r&d, luma labs gaussian splat renderer, looking glass etc. It would make sense to integrate it within the r3f doc for both discoverability - for those not on discord, or who rely a lot on exploring doc + github - and showcase, which could also in turn facilitate other teams/devs to add value to the fiber ecosystem.

Regarding maintenance/curation of the selection of projects to add to this page, this should probably be at the discretion of the pmndrs team which hosts the doc page, with community contributions for adding new projects or removing outdated ones suggested via PRs the same way anyone could contribute more codesandboxes examples to the example docs page.

I'm in favor of dissociating pages for

  • showcase + examples, the existing one - which often embed small components within the sandboxes themselves, but could also be stored on outside repos - great for demo/learning/reuse of utilities
  • from full-fledged projects like the 3D-tiles renderers, CustomShaderMaterial or the luma and looking-glass renderers, which are more complex, dedicated to a scoped use that will probably never be integrated into drei or a pmndrs repo.

Looking forward to hearing your thoughts!

@jo-chemla
Copy link
Author

Hi there, is there any feedback from others - either from the pmndrs team or outside - regarding this PR? I've integrated the feedback from @abernier to merge the community components into a section of the showcase page, but I'm open to extract it back to its own page if deemed useful.

Looking forward to hearing your feedback!

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

Successfully merging this pull request may close these issues.

2 participants