Skip to content

Conversation

@theodoregoetz
Copy link
Contributor

@theodoregoetz theodoregoetz commented Feb 11, 2025

Demonstration only!!! Do not merge to master.

This was tested against the dynamic-viewports3 branch in VulkanSceneGraph, commit: 1fac1761

The overlay views are partially off the screen. The initial rendering is good, but the model snaps to the scissors of the overlays when the window is resized. This is because the viewport gets set to the scissor in the viewportstate. It was expected that the viewport and the scissor remain independent and should be resized individually.

initial view (expected):

image

view after a window resize (the models should not be snapped to the scissors of the overlay views):

image

The overlay views are partially off the screen. The initial rendering is good, but the model snaps to the scissors of the overlays when the window is resized. This is because the viewport gets set to the scissor in the viewportstate. It was expected that the viewport and the scissor remain independent and should be resized individually.
@robertosfield
Copy link
Collaborator

For testing purpose I have merged this PR as a branch:

https://github.com/vsg-dev/vsgExamples/tree/theodoregoetz-vewport-partially-offscreen

On resize 'vsgcameras models/teapot.vsgt' resets the viewports to the extents of the window, which from the description of the issue is the problem that needs resolving.

There is also a bug in the management of the aspect ratio of the secondary views, this is a separate issue which occurs with vsgExamples master. I will be looking at both issues.

@robertosfield
Copy link
Collaborator

I have merged a refactor of WindowResizeHandler into VSG master that handles this usage case so that the modified vsgcameras example now works sensible:

vsg-dev/VulkanSceneGraph#1399

I'm going to close this PR as I believe it's no longer required.

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