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

Maintain shadow background during photo clipping #851

Conversation

patryk-sredzinski
Copy link
Contributor

I wanted to keep the shadow visible all the time during photo clipping. Got it working well.

We can add a property so the user can decide if he want to see it or not, but for now I've decided to leave it as it is.

Simulator.Screen.Recording.-.iPhone.14.Pro.Max.-.2023-08-31.at.13.13.58.mp4

@longitachi
Copy link
Owner

I think a property should be added to control this. I want the user to be able to clearly see all areas of the image when dragging the crop area.

@patryk-sredzinski patryk-sredzinski force-pushed the feature/maitain-shadow-background-during-clipping-photo branch from c790c31 to fa8c474 Compare August 31, 2023 15:19
@patryk-sredzinski
Copy link
Contributor Author

@longitachi I've added property called:

dimClippedAreaDuringAdjustments
defaults to false, to keep original functionality.

Tell me if you'd like me to do some other changes :)

@patryk-sredzinski patryk-sredzinski force-pushed the feature/maitain-shadow-background-during-clipping-photo branch from e97df6a to ea84b42 Compare September 1, 2023 14:39
@patryk-sredzinski patryk-sredzinski force-pushed the feature/maitain-shadow-background-during-clipping-photo branch from ea84b42 to d923820 Compare September 1, 2023 14:40
@patryk-sredzinski patryk-sredzinski force-pushed the feature/maitain-shadow-background-during-clipping-photo branch from 5fd1f97 to 8a69cdf Compare September 5, 2023 08:27
@longitachi longitachi merged commit 7086c8d into longitachi:master Sep 5, 2023
@patryk-sredzinski patryk-sredzinski deleted the feature/maitain-shadow-background-during-clipping-photo branch September 27, 2023 08:47
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.

None yet

2 participants