Skip to content

Fixed non rotatable devices rotating when "Rotate All" button in top of GUI is pressed#1439

Merged
manojVivek merged 4 commits intoresponsively-org:mainfrom
mklikushin:macbook-rotation
Oct 31, 2025
Merged

Fixed non rotatable devices rotating when "Rotate All" button in top of GUI is pressed#1439
manojVivek merged 4 commits intoresponsively-org:mainfrom
mklikushin:macbook-rotation

Conversation

@mklikushin
Copy link
Contributor

✨ Pull Request

📓 Referenced Issue

Fixes: #1438

ℹ️ About the PR

  • Added a device check on if the device is rotatable by verifying device.isTouchCapable || device.isMobileCapable
  • Added condition to if statement to only rotate device if the check is true.

🖼️ Testing Scenarios / Screenshots

Clicked the rotate devices button and Macbook Pro no longer rotates, new devices with no touch or mobile also do not rotate.

…utton-- rotation on individual devices still rotates regardless of device properties
@CLAassistant
Copy link

CLAassistant commented Oct 26, 2025

CLA assistant check
All committers have signed the CLA.

Copy link
Collaborator

@manojVivek manojVivek left a comment

Choose a reason for hiding this comment

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

Thank you for taking this up, just some minor feedback.

@mklikushin mklikushin requested a review from manojVivek October 27, 2025 08:55
@mklikushin
Copy link
Contributor Author

Please re-review @manojVivek

Copy link
Collaborator

@manojVivek manojVivek left a comment

Choose a reason for hiding this comment

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

Thank you for the followup, pushed a tiny commit and it is all looking good now! 🎉

@manojVivek manojVivek merged commit 8c055b5 into responsively-org:main Oct 31, 2025
3 of 4 checks passed
@manojVivek
Copy link
Collaborator

@all-contributors Please add @mklikushin for code.

@allcontributors
Copy link
Contributor

@manojVivek

I've put up a pull request to add @mklikushin! 🎉

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.

Non-rotating devices rotated by "rotate all" button in top of GUI.

3 participants