-
Notifications
You must be signed in to change notification settings - Fork 20
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 Translation Controls #529
Conversation
for more information, see https://pre-commit.ci
Preview PR at appsharing.space |
Integration tests repot: appsharing.space |
3bdfd84
to
8cc8601
Compare
6fad3a2
to
680f9cd
Compare
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
Integration tests report: appsharing.space |
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
After some testing I think this is looking and working great. Just one open question, do we want the transform control to be enabled for hidden objects? Happy to get your thoughts @arjxn-py @trungleduc @SylvainCorlay Allowing this allows that workflow: Screencast.From.2024-11-05.15-13-58.mp4 |
My first response to that would have been "No" but after looking at the workflow you've shared it looks like nice to have |
we might want to have an intermediate state for selected hidden object by playing with object's opacity. |
Same here. I saw it and though "cool!". |
I love it. Though transparency can be tricky, but if it's just one object at a time that should be fine. |
Just this remaining, if i'm not forgetting anything (for translation) |
This could be a separate PR. |
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.
Let's goooo! Thanks a lot for this, happy to see this go in
transform.mov
Shall partially supersede #151