Skip to content

Conversation

@enzofrancescaHM
Copy link

Description

the PR #20 has made superthree post-processing compatible with VR. Another PR has been opened in A-Frame (aframevr/aframe#5648) to create a minimal example to show bloom working in A-Frame in VR. A lot of hacks have been implemented in the JS of the example in order to make things work as intended, crucial part of them could be incorporated directly in super-three to lower the level of difficulty in user-land.

@dmarcos
Copy link
Member

dmarcos commented Feb 1, 2025

Thanks

@dmarcos dmarcos merged commit 4f4bcc4 into supermedium:dev Feb 1, 2025
8 of 10 checks passed
dmarcos pushed a commit that referenced this pull request Feb 1, 2025
* Update WebXRManager.js

* Update WebGLRenderer.js

* Update WebGLRenderer.js

deleted useless blank lines

* Update WebXRManager.js

- Deleted useless code and blank lines
- refectored _getRenderTarget to getRenderTarget

* Update WebGLRenderer.js

Reflect the refactoring of getRenderTarget function

Improve compatibility of Post-Processing with VR (#23)

* Update EffectComposer.js for better compatibility with VR

* Update Pass.js for better compatibility with VR
dmarcos pushed a commit that referenced this pull request Feb 2, 2025
* Update WebXRManager.js

* Update WebGLRenderer.js

* Update WebGLRenderer.js

deleted useless blank lines

* Update WebXRManager.js

- Deleted useless code and blank lines
- refectored _getRenderTarget to getRenderTarget

* Update WebGLRenderer.js

Reflect the refactoring of getRenderTarget function

Improve compatibility of Post-Processing with VR (#23)

* Update EffectComposer.js for better compatibility with VR

* Update Pass.js for better compatibility with VR
dmarcos pushed a commit that referenced this pull request Feb 3, 2025
* Update WebXRManager.js

* Update WebGLRenderer.js

* Update WebGLRenderer.js

deleted useless blank lines

* Update WebXRManager.js

- Deleted useless code and blank lines
- refectored _getRenderTarget to getRenderTarget

* Update WebGLRenderer.js

Reflect the refactoring of getRenderTarget function

Improve compatibility of Post-Processing with VR (#23)

* Update EffectComposer.js for better compatibility with VR

* Update Pass.js for better compatibility with VR
dmarcos pushed a commit that referenced this pull request Feb 28, 2025
* Update WebXRManager.js

* Update WebGLRenderer.js

* Update WebGLRenderer.js

deleted useless blank lines

* Update WebXRManager.js

- Deleted useless code and blank lines
- refectored _getRenderTarget to getRenderTarget

* Update WebGLRenderer.js

Reflect the refactoring of getRenderTarget function

Improve compatibility of Post-Processing with VR (#23)

* Update EffectComposer.js for better compatibility with VR

* Update Pass.js for better compatibility with VR
dmarcos pushed a commit that referenced this pull request Apr 3, 2025
* Update WebXRManager.js

* Update WebGLRenderer.js

* Update WebGLRenderer.js

deleted useless blank lines

* Update WebXRManager.js

- Deleted useless code and blank lines
- refectored _getRenderTarget to getRenderTarget

* Update WebGLRenderer.js

Reflect the refactoring of getRenderTarget function

Improve compatibility of Post-Processing with VR (#23)

* Update EffectComposer.js for better compatibility with VR

* Update Pass.js for better compatibility with VR
dmarcos pushed a commit that referenced this pull request Apr 3, 2025
* Update WebXRManager.js

* Update WebGLRenderer.js

* Update WebGLRenderer.js

deleted useless blank lines

* Update WebXRManager.js

- Deleted useless code and blank lines
- refectored _getRenderTarget to getRenderTarget

* Update WebGLRenderer.js

Reflect the refactoring of getRenderTarget function

Improve compatibility of Post-Processing with VR (#23)

* Update EffectComposer.js for better compatibility with VR

* Update Pass.js for better compatibility with VR
dmarcos pushed a commit that referenced this pull request Jun 14, 2025
* Update WebXRManager.js

* Update WebGLRenderer.js

* Update WebGLRenderer.js

deleted useless blank lines

* Update WebXRManager.js

- Deleted useless code and blank lines
- refectored _getRenderTarget to getRenderTarget

* Update WebGLRenderer.js

Reflect the refactoring of getRenderTarget function

Improve compatibility of Post-Processing with VR (#23)

* Update EffectComposer.js for better compatibility with VR

* Update Pass.js for better compatibility with VR
dmarcos pushed a commit that referenced this pull request Jun 14, 2025
* Update WebXRManager.js

* Update WebGLRenderer.js

* Update WebGLRenderer.js

deleted useless blank lines

* Update WebXRManager.js

- Deleted useless code and blank lines
- refectored _getRenderTarget to getRenderTarget

* Update WebGLRenderer.js

Reflect the refactoring of getRenderTarget function

Improve compatibility of Post-Processing with VR (#23)

* Update EffectComposer.js for better compatibility with VR

* Update Pass.js for better compatibility with VR
dmarcos pushed a commit that referenced this pull request Oct 21, 2025
* Update WebXRManager.js

* Update WebGLRenderer.js

* Update WebGLRenderer.js

deleted useless blank lines

* Update WebXRManager.js

- Deleted useless code and blank lines
- refectored _getRenderTarget to getRenderTarget

* Update WebGLRenderer.js

Reflect the refactoring of getRenderTarget function

Improve compatibility of Post-Processing with VR (#23)

* Update EffectComposer.js for better compatibility with VR

* Update Pass.js for better compatibility with VR
dmarcos pushed a commit that referenced this pull request Nov 12, 2025
* Update WebXRManager.js

* Update WebGLRenderer.js

* Update WebGLRenderer.js

deleted useless blank lines

* Update WebXRManager.js

- Deleted useless code and blank lines
- refectored _getRenderTarget to getRenderTarget

* Update WebGLRenderer.js

Reflect the refactoring of getRenderTarget function

Improve compatibility of Post-Processing with VR (#23)

* Update EffectComposer.js for better compatibility with VR

* Update Pass.js for better compatibility with VR
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