Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RenderFrontEnd #3319

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from
Open

RenderFrontEnd #3319

wants to merge 4 commits into from

Conversation

Geokureli
Copy link
Member

@Geokureli Geokureli commented Dec 23, 2024

  • Adds FlxG.render.pixelMode. Possible values:
    • PIXELATED: Sets all rendering options to be pixel-perfect
    • SMOOTH: Enabled anti-Aliasing, disables pixel-perfect
    • CUSTOM: The default value, changes nothing
  • Adds FlxG.render.tile and blit to replace FlxG.renderTile and renderBlit

To Do

  • Rethink these names
  • add a pixel shader?

@Geokureli Geokureli changed the title DisplayFrontEnd RenderFrontEnd Dec 23, 2024
@ACrazyTown
Copy link
Contributor

ACrazyTown commented Dec 23, 2024

Maybe renderTile could be renamed to renderQuads or something as the old name is a remnant from when Flixel used OpenFL's tilesheet for rendering (could also rename the flixel.graphics.tile package to something like flixel.graphics.render or flixel.graphics.draw for consistency)

@Geokureli
Copy link
Member Author

Geokureli commented Dec 23, 2024

Maybe renderTile could be renamed to renderQuads or something as the old name is a remnant from when Flixel used OpenFL's tilesheet for rendering (could also rename the flixel.graphics.tile package to something like flixel.graphics.render or flixel.graphics.draw for consistency)

It was actually requested that I look into openfl's tilemaps, as they have been working on them as of late

Also I think "quad" isn't accurate because it can either be quads or triangles

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