-
Notifications
You must be signed in to change notification settings - Fork 31
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
Modal Refinements Update #144
Modal Refinements Update #144
Conversation
Pull Request Test Coverage Report for Build 1068904178Details
💛 - Coveralls |
…& Export Settings
…aspect ratio are shown
@chrisgervang Few comments:
|
Sorry for the confusion. That note was from the previous PR that didn't get merged in. I removed it but the reason Export isn't the default anymore was that we decided that we wanted the flow to be to allow users to select the type of animation before picking settings like resolution and aspect ratio. The currently active tab in that screenshot is Animation. I think that addresses your concern but let me know if you have more questions. |
|
Updated version of this PR. Previous PR was not merged and now touches parts of the codebase I didn't write.
Changelog