Update PanCam
and FreeCam
to use full term Camera
#21592
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Objective
This PR addresses #21569, which proposes renaming the newly introduced camera controller modules
FreeCam
andPanCam
to use the full termCamera
.This PR resolves the
PanCam
portion.Solution
PanCam
controller,PanCamPlugin
, and related methods to use the full termCamera
instead of the abbreviationCam
.pan_cam
topan_camera
for consistency with naming conventions.pan_camera_controller
and adjusted usage of the renamed controller and plugin.Follow-up Work
I see two options from here:
Use this PR as a reference for renaming
FreeCam
. The process is similar and could be a great first issue for someone looking to contribute to the new camera modules or Bevy in general. Most of the changes follow the same pattern, althoughFreeCam
has more examples that need updating. One could find them usinggrep
(e.g.,grep FreeCam
) or by reviewing the diff from the PR that introducedFreeCam
: Add bevy_camera_controllers crate and move freecam implementation into it #20215I can continue and update this PR to also handle the
FreeCam
renaming, if you'd prefer to resolve the entire issue in one go.