Skip to content

FlxCamera: make the filters array public#2890

Merged
Geokureli merged 8 commits intoHaxeFlixel:devfrom
Sword352:camera-filters-extension
Aug 15, 2023
Merged

FlxCamera: make the filters array public#2890
Geokureli merged 8 commits intoHaxeFlixel:devfrom
Sword352:camera-filters-extension

Conversation

@Sword352
Copy link
Copy Markdown
Contributor

This pull request add the addFilter and removeFilter functions to FlxCamera.
Should be useful for many situations

@Sword352
Copy link
Copy Markdown
Contributor Author

Might as well add functions for shaders later on since they are used a lot (by using ShaderFilter)

@Geokureli
Copy link
Copy Markdown
Member

as i mention in #2888, I think we should rethink filters. if you're gonna add an add and remove function maybe we should just make this var public

@Sword352
Copy link
Copy Markdown
Contributor Author

Yeah that make sense to just make it public, let me do a little commit

@Sword352
Copy link
Copy Markdown
Contributor Author

Sword352 commented Aug 13, 2023

Alright, the _filters array is now deprecated in favor of the public filters array. setFilters and getFilters are deprecated as well, addFilter and removeFilter are now only 2 helper functions to add and remove filters safely into the camera.

Comment thread flixel/FlxCamera.hx Outdated
Comment thread flixel/FlxCamera.hx Outdated
Comment thread flixel/FlxCamera.hx Outdated
@Sword352
Copy link
Copy Markdown
Contributor Author

All done

@Sword352 Sword352 closed this Aug 13, 2023
@Sword352 Sword352 reopened this Aug 13, 2023
@Sword352
Copy link
Copy Markdown
Contributor Author

Accidentally closed the pr 😅

@Sword352 Sword352 changed the title Add a bunch of filter functions for FlxCamera FlxCamera: make the filters array public Aug 13, 2023
Copy link
Copy Markdown
Member

@Geokureli Geokureli left a comment

Choose a reason for hiding this comment

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

some more changes

Comment thread flixel/FlxCamera.hx Outdated
Comment thread flixel/FlxCamera.hx Outdated
@Sword352
Copy link
Copy Markdown
Contributor Author

Done as well

Copy link
Copy Markdown
Member

@Geokureli Geokureli left a comment

Choose a reason for hiding this comment

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

sorry couple more things

Comment thread flixel/FlxCamera.hx Outdated
Comment thread flixel/FlxCamera.hx Outdated
@Sword352
Copy link
Copy Markdown
Contributor Author

Done

@Geokureli Geokureli merged commit ab3dbfc into HaxeFlixel:dev Aug 15, 2023
@Geokureli
Copy link
Copy Markdown
Member

Geokureli commented Aug 15, 2023

Sorry for the delay I had a busy weekend. Thanks for being patient, @Sword352

@Sword352 Sword352 deleted the camera-filters-extension branch August 15, 2023 17:05
charlesisfeline pushed a commit to AumSum-Funkin-Team/PlutosReprisalAumSumTakeover that referenced this pull request Oct 17, 2023
- Remove `FlxRuntimeShader` since Haxeflixel added `FlxRuntimeShader` HaxeFlixel/flixel-addons#399
* Migrate `FlxAnimationController` to Flixel 5.4.1 HaxeFlixel/flixel#2930
* Fix `FlxCamera.filters` Deprecation Warning HaxeFlixel/flixel#2890
@Geokureli Geokureli added this to the 5.4.0 milestone Jan 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants