Skip to content

feat/Add ScanlineEffect & demo to v7 - #602

Merged
vanruesc merged 14 commits into
pmndrs:v7from
balraj-johal:v7-scanlines-demo
Feb 20, 2024
Merged

vanruesc merged 14 commits into
pmndrs:v7from
balraj-johal:v7-scanlines-demo

Conversation

@balraj-johal

Copy link
Copy Markdown

Related issue: #595

Description

This PR ports the v6 implementation of the ScanlineEffect to the v7 alpha build. Changes made were primarily modelled after the v7 version of the VignetteEffect.

Comment thread src/effects/ScanlineEffect.ts Resolved
Comment thread src/effects/ScanlineEffect.ts Resolved

@balraj-johal balraj-johal left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have commented a few questions I had :)

@balraj-johal
balraj-johal marked this pull request as ready for review February 20, 2024 19:53
@balraj-johal balraj-johal mentioned this pull request Feb 20, 2024
3 tasks done
Comment thread manual/assets/js/src/demos/scanlines.ts Resolved
Comment thread manual/assets/js/src/demos/scanlines.ts Resolved
Comment thread src/effects/ScanlineEffect.ts Resolved
Comment thread src/effects/ScanlineEffect.ts Resolved
Comment thread src/effects/ScanlineEffect.ts Resolved
Comment thread src/effects/ScanlineEffect.ts Resolved
Comment thread src/effects/ScanlineEffect.ts Resolved
Comment thread src/effects/ScanlineEffect.ts Resolved
Comment thread src/effects/ScanlineEffect.ts Resolved
Comment thread src/effects/index.ts Resolved
@balraj-johal

Copy link
Copy Markdown
Author

@vanruesc thanks for the review! I think I've addressed all your comments :)

Comment thread src/effects/ScanlineEffect.ts Resolved
@vanruesc
vanruesc merged commit 8c59fee into pmndrs:v7 Feb 20, 2024
@vanruesc

Copy link
Copy Markdown
Member

Thank you!

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