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.

Sign up for GitHub

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

Jump to bottom

Remove Offset in Static crop type #4

Closed
ceyton opened this issue Oct 9, 2022 · 3 comments
Closed

Remove Offset in Static crop type #4

ceyton opened this issue Oct 9, 2022 · 3 comments

Comments

@ceyton
Copy link

ceyton commented Oct 9, 2022

Is it possible to remove the Offset in the Static crop type?

image
@SmartToolFactory
Copy link
Owner

SmartToolFactory commented Oct 10, 2022 •
edited

Other croppers have this offset too
but i can add a flag to remove offset. Also i think i should adjust zoom to be less than 1f match drawing area to be able to crop with no offset on any sides if needed.

@ceyton
Copy link
Author

ceyton commented Oct 10, 2022 •
edited

Other croppers have this offset too but i can add a flag to remove offset. Also i think i should adjust zoom to be less than 1f match drawing area to be able to crop with no offset on any sides if needed.

in most cases, I think of, I don't think that scale should be less than 1f, regarding the offset I was trying to implement Instagram-like cropping this is why it would really be nice to have it without the offset 👍

@SmartToolFactory SmartToolFactory mentioned this issue Oct 13, 2022
Closed
SmartToolFactory added a commit that referenced this issue Oct 15, 2022
@SmartToolFactory
add option to change overlay ratio to screen #4
7c6c9b0 
* Adds option to change overlay ratio to screen dimension with smaller size. If width is smaller than height percentage of width is used to scale overlay

* Adjusts style of TextFiled and Sliders for better settings UI
@SmartToolFactory
Copy link
Owner

Overlay ratio option in settings menu can scale overlay between 50 and 100%.

overlay_ratio

SmartToolFactory pushed a commit that referenced this issue May 29, 2023
@Nimrodda
Merge pull request #4 from Nimrodda/agp-801
b8ed114 
Updated AGP to version 8.0.1
@nullsafety nullsafety mentioned this issue Dec 19, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@SmartToolFactory @ceyton