Skip to content

Conversation

@CodyJasonBennett
Copy link
Member

@CodyJasonBennett CodyJasonBennett commented Apr 4, 2023

Sets gl.outputColorSpace (mrdoob/three.js#25756) and Texture.colorSpace (mrdoob/three.js#25771) since r152.

This is one of the last relevant changes aside from switching to sRGB by default, and maybe disabling legacy lighting (see mrdoob/three.js#23614).

@codesandbox-ci
Copy link

codesandbox-ci bot commented Apr 4, 2023

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 a4ba957:

Sandbox Source
example Configuration

@CodyJasonBennett CodyJasonBennett merged commit d7d8907 into master Apr 27, 2023
@CodyJasonBennett CodyJasonBennett deleted the fix/r152-output-colorspace branch April 27, 2023 10:45
renovate bot referenced this pull request in ziyadedher/ziyadedher Apr 28, 2023
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [@react-three/fiber](https://togithub.com/pmndrs/react-three-fiber) |
[`8.12.2` ->
`8.13.0`](https://renovatebot.com/diffs/npm/@react-three%2ffiber/8.12.2/8.13.0)
|
[![age](https://badges.renovateapi.com/packages/npm/@react-three%2ffiber/8.13.0/age-slim)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://badges.renovateapi.com/packages/npm/@react-three%2ffiber/8.13.0/adoption-slim)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://badges.renovateapi.com/packages/npm/@react-three%2ffiber/8.13.0/compatibility-slim/8.12.2)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://badges.renovateapi.com/packages/npm/@react-three%2ffiber/8.13.0/confidence-slim/8.12.2)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>pmndrs/react-three-fiber</summary>

###
[`v8.13.0`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.13.0)

[Compare
Source](https://togithub.com/pmndrs/react-three-fiber/compare/@react-three/fiber@8.12.2...v8.13.0)

#### What's Changed

- fix: set outputColorSpace with r152 by
[@&#8203;CodyJasonBennett](https://togithub.com/CodyJasonBennett) in
[https://github.com/pmndrs/react-three-fiber/pull/2829](https://togithub.com/pmndrs/react-three-fiber/pull/2829)
- feat(Canvas): alias Props => CanvasProps by
[@&#8203;CodyJasonBennett](https://togithub.com/CodyJasonBennett) in
[https://github.com/pmndrs/react-three-fiber/pull/2842](https://togithub.com/pmndrs/react-three-fiber/pull/2842)

#### New Contributors

- [@&#8203;nscharpff](https://togithub.com/nscharpff) made their first
contribution in
[https://github.com/pmndrs/react-three-fiber/pull/2807](https://togithub.com/pmndrs/react-three-fiber/pull/2807)
- [@&#8203;brian3814](https://togithub.com/brian3814) made their first
contribution in
[https://github.com/pmndrs/react-three-fiber/pull/2819](https://togithub.com/pmndrs/react-three-fiber/pull/2819)
- [@&#8203;tometo-dev](https://togithub.com/tometo-dev) made their first
contribution in
[https://github.com/pmndrs/react-three-fiber/pull/2813](https://togithub.com/pmndrs/react-three-fiber/pull/2813)
- [@&#8203;adamaslan](https://togithub.com/adamaslan) made their first
contribution in
[https://github.com/pmndrs/react-three-fiber/pull/2838](https://togithub.com/pmndrs/react-three-fiber/pull/2838)
- [@&#8203;RakaDoank](https://togithub.com/RakaDoank) made their first
contribution in
[https://github.com/pmndrs/react-three-fiber/pull/2847](https://togithub.com/pmndrs/react-three-fiber/pull/2847)
- [@&#8203;abernier](https://togithub.com/abernier) made their first
contribution in
[https://github.com/pmndrs/react-three-fiber/pull/2843](https://togithub.com/pmndrs/react-three-fiber/pull/2843)

**Full Changelog**:
pmndrs/react-three-fiber@v8.12.0...v8.13.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://app.renovatebot.com/dashboard#github/ziyadedher/ziyadedher).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNS42Mi4wIiwidXBkYXRlZEluVmVyIjoiMzUuNjIuMCJ9-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
ziyadedher referenced this pull request in ziyadedher/ziyadedher Dec 16, 2023
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [@react-three/fiber](https://togithub.com/pmndrs/react-three-fiber) |
[`8.12.2` ->
`8.13.0`](https://renovatebot.com/diffs/npm/@react-three%2ffiber/8.12.2/8.13.0)
|
[![age](https://badges.renovateapi.com/packages/npm/@react-three%2ffiber/8.13.0/age-slim)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://badges.renovateapi.com/packages/npm/@react-three%2ffiber/8.13.0/adoption-slim)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://badges.renovateapi.com/packages/npm/@react-three%2ffiber/8.13.0/compatibility-slim/8.12.2)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://badges.renovateapi.com/packages/npm/@react-three%2ffiber/8.13.0/confidence-slim/8.12.2)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>pmndrs/react-three-fiber</summary>

###
[`v8.13.0`](https://togithub.com/pmndrs/react-three-fiber/releases/tag/v8.13.0)

[Compare
Source](https://togithub.com/pmndrs/react-three-fiber/compare/@react-three/fiber@8.12.2...v8.13.0)

#### What's Changed

- fix: set outputColorSpace with r152 by
[@&#8203;CodyJasonBennett](https://togithub.com/CodyJasonBennett) in
[https://github.com/pmndrs/react-three-fiber/pull/2829](https://togithub.com/pmndrs/react-three-fiber/pull/2829)
- feat(Canvas): alias Props => CanvasProps by
[@&#8203;CodyJasonBennett](https://togithub.com/CodyJasonBennett) in
[https://github.com/pmndrs/react-three-fiber/pull/2842](https://togithub.com/pmndrs/react-three-fiber/pull/2842)

#### New Contributors

- [@&#8203;nscharpff](https://togithub.com/nscharpff) made their first
contribution in
[https://github.com/pmndrs/react-three-fiber/pull/2807](https://togithub.com/pmndrs/react-three-fiber/pull/2807)
- [@&#8203;brian3814](https://togithub.com/brian3814) made their first
contribution in
[https://github.com/pmndrs/react-three-fiber/pull/2819](https://togithub.com/pmndrs/react-three-fiber/pull/2819)
- [@&#8203;tometo-dev](https://togithub.com/tometo-dev) made their first
contribution in
[https://github.com/pmndrs/react-three-fiber/pull/2813](https://togithub.com/pmndrs/react-three-fiber/pull/2813)
- [@&#8203;adamaslan](https://togithub.com/adamaslan) made their first
contribution in
[https://github.com/pmndrs/react-three-fiber/pull/2838](https://togithub.com/pmndrs/react-three-fiber/pull/2838)
- [@&#8203;RakaDoank](https://togithub.com/RakaDoank) made their first
contribution in
[https://github.com/pmndrs/react-three-fiber/pull/2847](https://togithub.com/pmndrs/react-three-fiber/pull/2847)
- [@&#8203;abernier](https://togithub.com/abernier) made their first
contribution in
[https://github.com/pmndrs/react-three-fiber/pull/2843](https://togithub.com/pmndrs/react-three-fiber/pull/2843)

**Full Changelog**:
pmndrs/react-three-fiber@v8.12.0...v8.13.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://app.renovatebot.com/dashboard#github/ziyadedher/ziyadedher).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNS42Mi4wIiwidXBkYXRlZEluVmVyIjoiMzUuNjIuMCJ9-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
bigmistqke added a commit to solidjs-community/solid-three that referenced this pull request Apr 28, 2024
LinearEncoding and sRGBEncoding is not included anymore in later versions of threejs. react-three-fiber hardcoded these enums.

see pmndrs/react-three-fiber#2829
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