-
-
Notifications
You must be signed in to change notification settings - Fork 17
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 mirror the webcam image #138
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.
I think it would make sense to wrap this row in a modelbutton like we do in Code/Terminal settings popovers, so you can click anywhere in the row to toggle it.
Co-authored-by: Cassidy James Blaede <cassidy@elementary.io>
@cassidyjames You mean like this? I am having a hard time aligning the button, but at least this looks like an improvement over the initial version. What do you think? |
@igordsm I agree with Cassidy, You could use a ModeButton |
Well, that's embarrassing. It was ModeButton and not ModelButton. Sorry about that. I'll try and fix it. |
@igordsm no, I did mean ModelButton, just with a switch—like we use in the Terminal settings popover. 😄 |
@cassidyjames 😆 You mean like this? If you do, then this is ready for review. |
@igordsm yep, that's much closer. 😁 |
@cassidyjames I added |
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.
Looks really good, just a few comments :)
Co-authored-by: Daniel Foré <daniel@elementary.io>
Co-authored-by: Cassidy James Blaede <cassidy@elementary.io>
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.
Thanks for being patient with all of the requested changes! This is looking good. Happy to merge 🎉
Fix #38 : add option to mirror image.
Following the discussions on #67 , IMHO lacking this kind of feature may take users away from using camera. Specially when recording videos, being able to do simple adjustments directly from the app would make it infinitely more useful for me. I initially implemented it using a simple button, mas changed it to an app menu so that #51 could be implemented as well (I have a prototype, in fact).
What do you think? If you think its better to fix it one at a time I could change this one to use a "object-flip-horizontal" button, like below.