Skip to content

feat: add ability to disable camera rotation#472

Open
DaNyNaNd wants to merge 1 commit intoApparence-io:masterfrom
Redspur:feature/disable-rotation
Open

feat: add ability to disable camera rotation#472
DaNyNaNd wants to merge 1 commit intoApparence-io:masterfrom
Redspur:feature/disable-rotation

Conversation

@DaNyNaNd
Copy link
Copy Markdown

@DaNyNaNd DaNyNaNd commented May 8, 2024

Description

Add the ability to turn camera rotation on and off for a photo

Checklist

Before creating any Pull Request, confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).

  • 📕 I read the Contributing page.
  • 🤝 I match the actual coding style.
  • ✅ I ran flutter analyze without any issues.

Breaking Change

  • 🛠 My feature contain breaking change.

If your feature break something, please detail it

@DaNyNaNd DaNyNaNd closed this May 8, 2024
@DaNyNaNd DaNyNaNd reopened this May 8, 2024
@cartworthy
Copy link
Copy Markdown

Hi, I'm currently using camerAwesome on my flutter project and everything works except after capturing landscape images they are being saved in the portrait orientation. Do you know how to solve this problem? I'd be willing to pay for your help on this issue.

Please contact josh@cartworthy.co if you're able to help, thanks!

@DaNyNaNd
Copy link
Copy Markdown
Author

@cartworthy If you're working off the branch from this PR, then it's set to only take images in portrait mode if you set enableRotation: false in the class instantiation. Otherwise, it defaults to true, so you can leave the enableRotation property off.

@cartworthy
Copy link
Copy Markdown

@DaNyNaNd Hi! Thanks for getting back to me. I'm not using this PR and I'm not a developer — I just found this from a Google Search looking for "disable rotation CamerAwesome" and thought this may do what I need. I'm building the app with FlutterFlow (no-code platform) and use camerAwesome for a custom widget.

Would you be able to hop on a video call to help me troubleshoot this? I'll venmo you $20 :)

@DaNyNaNd
Copy link
Copy Markdown
Author

@g-apparence do you see any chance this PR will get merged? Or, is there functionality in the updated package that would make this not needed?

@cartworthy
Copy link
Copy Markdown

@DaNyNaNd I tried editing the code to include "enableRotation: false" but it didn't have any effect.

I'd be happy to venmo you $100 if you be willing to hop on a video call just to tell me where to paste it haha. I am desperate for a solution to this. Please email me if you're interested in helping: josh@cartworthy.co

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.

2 participants