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

Add PTZ support for flipping the canvas #2394

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

Conversation

Sidharth-Singh10
Copy link
Contributor

Closes #1722

@Sidharth-Singh10 Sidharth-Singh10 marked this pull request as ready for review March 7, 2025 09:41
@Keavon Keavon marked this pull request as draft March 7, 2025 09:49
@Sidharth-Singh10 Sidharth-Singh10 marked this pull request as ready for review March 9, 2025 01:48
Copy link
Member

@0HyperCube 0HyperCube left a comment

Choose a reason for hiding this comment

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

Thanks for your patience with this. It appears to work well and the code looks good.

I'll leave it to @Keavon if they wish to QA test as well. Also we probably need to think about the widget design for this.

@0HyperCube 0HyperCube requested a review from Keavon March 9, 2025 11:55
Copy link

github-actions bot commented Mar 9, 2025

📦 Build Complete for f4b4646
https://d3218326.graphite.pages.dev

@Keavon Keavon changed the title Flip Canvas Add PTZ support for flipping the canvas Mar 9, 2025
@Keavon
Copy link
Member

Keavon commented Mar 9, 2025

I'll try to work on the UI within the next couple of days. The feature looks like it's all working now, although I'll do more extensive testing when I'm adding the UI change.

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.

"Flip" view navigation button
3 participants