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

Add camera view editing mode #100

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

khanghugo
Copy link
Contributor

Closes #96

I think the organizing is a bit weird. Not sure how to go around that because this needs the game itself to work.

Need YaLTeR/BunnymodXT#520 to have it work.

src/modules/tas_studio/editor/mod.rs Outdated Show resolved Hide resolved
src/modules/tas_studio/editor/mod.rs Outdated Show resolved Hide resolved
// Mouse was released last frame so the adjustment cannot be active.
assert!(self.camera_view_adjustment.is_none());

// There are no other adjustments in the camera editor at the moment; and anyhow
Copy link
Owner

Choose a reason for hiding this comment

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

I guess this comment needs an update here and in the other place

Copy link
Contributor Author

Choose a reason for hiding this comment

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

What kind of comment change do you prefer here? Straight up deletion?

@@ -296,6 +305,23 @@ struct InsertCameraLineAdjustment {
did_split: bool,
}

/// Camera view edit modes.
///
/// `Alt` adjusts differently for different kinds of line.
Copy link
Owner

Choose a reason for hiding this comment

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

Could you describe here what exactly it does?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Alt is in case you want the result to be something like target yaw offset or target yaw velocity lock.

src/modules/tas_studio/editor/mod.rs Outdated Show resolved Hide resolved
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.

TAS Editor Camera Editor Camera Editting Mode
2 participants