Giter VIP home page Giter VIP logo

Comments (5)

KylePalko avatar KylePalko commented on August 22, 2024 1

Sweet, thanks for the direction, I will take a look. Again, appreciate all the work you've put into this!

from react-pixi-fiber.

michalochman avatar michalochman commented on August 22, 2024

Hi @KylePalko, it should be fairly easy to fix for server-side rendering by lazy loading the Stage (this is now very easy with React 16.6+) and rendering it behind a flag that checks if the render is within browser context. The other issue is that PixiJS itself probably needs a window object so it would need to be lazily imported as well.

However, I understand this might be an unnecessary boilerplate so I am open to receiving a pull request that solves this without having to change the app code. See #80 for more context. Please note that stubbing Stage to return children is probably not the best idea – for SSR it would be better to just render the canvas element with its props and skip the logic related to instantiating PixiJS Application and react-pixi-fiber rendering.

from react-pixi-fiber.

KylePalko avatar KylePalko commented on August 22, 2024

React.lazy isn't available for SSR yet :/ The React docs point towards using Loadable Components module though. Is that what you were imagining?

for SSR it would be better to just render the canvas element with its props and skip the logic related to instantiating PixiJS Application and react-pixi-fiber rendering.

Can you give me a bit of direction on how this might work? How would I render the canvas element server-side? You're blowing my mind here man XD

EDIT: Also, thanks for all your hard work, this module is awesome!

from react-pixi-fiber.

michalochman avatar michalochman commented on August 22, 2024

@KylePalko ah, you're right, lazy is not in SSR yet.

How would I render the canvas element server-side

Stage already renders canvas element so probably nothing to change here.

I think all that's really needed is:

  1. Not loading PIXI if not in browser context. This would probably mean changing this import to require behind if condition.
  2. Changing Stage's componentDidMount to no-op if not in browser context.

from react-pixi-fiber.

michalochman avatar michalochman commented on August 22, 2024

See if the solution in this comment fits.

from react-pixi-fiber.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.