Skip to content

Buffer Resize #187

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 11 commits into
base: main
Choose a base branch
from
Draft

Buffer Resize #187

wants to merge 11 commits into from

Conversation

@ReimousTH
Copy link
Contributor

@ReimousTH ReimousTH commented Nov 1, 2025 •
edited
Loading

TODO:

  • Reflections are broken
  • Radarmap does not scale
  • CSM and character rendering breaks when resize on pasue
  • Main Menu MST Text Resize

@IsaacMarovitz IsaacMarovitz marked this pull request as draft November 1, 2025 21:49
@IsaacMarovitz IsaacMarovitz changed the title Window resize Buffer Resize Nov 1, 2025
hyperbx
hyperbx reviewed Nov 3, 2025
View reviewed changes
ReimousTH and others added 8 commits November 7, 2025 20:43
@ReimousTH
Graphics Api & Resize Base
c298405
@ReimousTH
After
f1d826d
@IsaacMarovitz @ReimousTH
Cleanup
539b0a7 
Signed-off-by: Isaac Marovitz <[email protected]>
@ReimousTH
radermap & radermap_mask resize fix
ff91c1d 
radermap now do properly resize
depthstencil_1_4 does not support resize otherwise may cause issues reflection(...surface cache)
@hyperbx @ReimousTH
Clean-up, enable gfx options, use pre-existing window resize notifica…
0856116 
…tions
@IsaacMarovitz @ReimousTH
Fix compiler differences
6d26eab 
Signed-off-by: Isaac Marovitz <[email protected]>
@ReimousTH
MyPE & Particles & Spanverse API
36e9aee 
More Api
Manual Lua Reset
@ReimousTH
Proper WindowsResize
c5221cd 
Particles & CSM & Reflection & Hint Text & Combo Text
TODO : Proper Main Menu MST Text during Resize
@hyperbx hyperbx added the blocked This should not be merged label Nov 8, 2025
@hyperbx
Copy link
Collaborator

hyperbx commented Nov 8, 2025

This PR has a memory leak due to an underlying issue with the GPU backend that should be addressed first.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hyperbx hyperbx hyperbx left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

blocked This should not be merged

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ReimousTH @hyperbx @IsaacMarovitz