Skip to content

Add PTZ support for flipping the canvas #2394

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

Merged
merged 6 commits into from
May 18, 2025

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.

@Keavon Keavon force-pushed the master branch 4 times, most recently from aa7ff13 to e11b57a Compare April 6, 2025 11:41
@Keavon
Copy link
Member

Keavon commented May 18, 2025

All done! Sorry this took a while to tend to.

@Keavon Keavon enabled auto-merge (squash) May 18, 2025 06:10
@Keavon Keavon merged commit 2615d86 into GraphiteEditor:master May 18, 2025
5 checks passed
@Keavon
Copy link
Member

Keavon commented May 20, 2025

@Sidharth-Singh10 just a reminder to please comment with this PR number in its issue, #1722, so I can assign that to you for the credit! Thanks :)

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