Skip to content

Conversation

@MrStevns
Copy link
Member

@MrStevns MrStevns commented Oct 24, 2022

UI change:

  • Handles are now white with thicker outline, to differentiate better between active/disabled state

Active:
image
Disabled:
image

Fixes:

  • Only show skinning under playback if enabled
  • Camera handles are only shown while camera tool is active
  • Camera interpolation dots are only shown while camera tool is active
  • Camera field is not shown during playback
  • Fix onion skinning was applied per keyframe.

- Only show skinning under playback if enabled
- Don't paint handles unless we can interact with them
- Fix onion skinning was applied per keyframe.
- Only draw interpolations while handles are active
@J5lx J5lx added Bug Camera UI Related to the visual appearance of the program labels Oct 24, 2022
@J5lx J5lx self-assigned this Oct 24, 2022
Copy link
Member

@J5lx J5lx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good all in all, there are only two small issues I found:

  • the interpolation dots are now shown during playback.
  • the inactive handles blend into the background too much. It does make sense to make them more subtle, but this is a bit too much.

Also, how come you removed the dashed lines to the control points? I thought they were a useful visual aid.

But yeah, that aside the PR looks good!

@MrStevns
Copy link
Member Author

the interpolation dots are now shown during playback.

This is intentional, given that they're only visible now when: you're on a camera layer and the "show paths" are toggled on. It's not related to onion skinning, so it is not affected by "Show during playback"

the inactive handles blend into the background too much. It does make sense to make them more subtle, but this is a bit too much.

Fair point, i'll tweak it.

Also, how come you removed the dashed lines to the control points? I thought they were a useful visual aid.

I don't think the dashed line provide any meaningful addition to the UI when you have the dots to represent the curve but if you think it's useful, i'll bring it back.

Copy link
Member

@J5lx J5lx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for your work!

@J5lx J5lx merged commit a352e19 into pencil2d:master Oct 29, 2022
@MrStevns MrStevns added this to the v0.6.7 milestone Jul 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Camera UI Related to the visual appearance of the program

Projects

Status: No status
Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants