Skip to content

[v10] Remove Flat in favor of actual tonemapping. #3612

@DennisSmolek

Description

@DennisSmolek

With #3610 and other cleanup I think we can drop the "flat" prop as well.

<Canvas flat > //old
<Canvas tonemapping={THREE.NoToneMapping} > // direct

Yes its more verbose but its intentional and wont be confusing when people try to do:

<Canvas legacy flat tonemapping={THREE.AcesTonemapping} >

Or other nightmares I've seen. Its unclear with those prop tags which has priority and what will actually happen to color. This way its very clear whats doing what

Metadata

Metadata

Assignees

Labels

colorRelated to color, colorSpace, tonemapping etcenhancementNew feature or requestv10

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions