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

Option to flip back of board 180º #332

Merged
merged 4 commits into from
Oct 18, 2022
Merged

Conversation

dotsam
Copy link
Contributor

@dotsam dotsam commented Oct 15, 2022

Fixes #328

I'm not totally sold on "Flip Back" as the friendly/user-facing name of this feature, so I'm open to suggestions on other things to call it.

@qu1ck
Copy link
Member

qu1ck commented Oct 16, 2022

Yeah, flip usually means back to front (and is used this way in code too), not in plane rotation. I think "Offset back rotation" is better.

Code looks great. Only thing: the checkbox in html settings looks wedged. Put it in the same box as the rotation slider, right under it. Basically like it looks in the config dialog.

@dotsam
Copy link
Contributor Author

dotsam commented Oct 17, 2022

Updated with new display name and variables names everywhere, as well as the HTML layout change.

Will you squash on merge, or would you like me to force-push a single commit for all of this?

@dotsam dotsam changed the title Option to flip back on board 180º Option to flip back of board 180º Oct 17, 2022
@qu1ck qu1ck merged commit 4e53f4e into openscopeproject:master Oct 18, 2022
@qu1ck
Copy link
Member

qu1ck commented Oct 18, 2022

Squashed and merged, thanks for contributing.

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.

Allow rotating front/back separately/inversely
2 participants