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 option to choose to take a screenshot with the device dimensions being the minimum. #1058
base: main
Are you sure you want to change the base?
Add option to choose to take a screenshot with the device dimensions being the minimum. #1058
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a very thoughtful feature and glad you implemented it!
Just a couple of minor thoughts, looks great otherwise!
@@ -148,6 +148,10 @@ const schema = { | |||
}, | |||
default: {}, | |||
}, | |||
onlyVisibleParts: { | |||
type: 'boolean', | |||
default: false, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we make the default as true
? So that the current behaviour stays the same unless specifically modified in the settings.
|
||
try { | ||
if (rect != null) { | ||
console.log(rect); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please remove this debug log line?
screenShotDelegate.setContentSize(rect.width, rect.height); | ||
} | ||
} catch (ex) { | ||
// |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe console.log the error here?
✨ Pull Request
📓 Referenced Issue
ℹ️ About the PR
This is useful when you want to take a screenshot with the device dimensions being the minimum.
🖼️ Testing Scenarios / Screenshots
With:
Without: