Skip to content

Conversation

@Shengle-Dai
Copy link
Collaborator

Summary

Implemented the rotating cursor feature—updating the direction of the marker/cursor based on the user's magnetic heading direction.

  • fixed dependency issues on the local machine
  • implemented the core functionality to update the cursor rotation with the device's magnetic heading.

Test Plan

  • Simulate different compassHeading values to verify the cursor’s update.
  • Test the feature on TestFlight to ensure performance.

@CLAassistant
Copy link

CLAassistant commented Oct 27, 2024

CLA assistant check
All committers have signed the CLA.

@dti-github-bot
Copy link
Member

dti-github-bot commented Oct 27, 2024

[diff-counting] Significant lines: 24.

@3TTemi
Copy link
Collaborator

3TTemi commented Oct 28, 2024

Looks good! I think this solution will work.
However, I would suggest adding a check for null values or defaulting the rotation to currentLocation.heading, as was used before. Additionally, I noticed that you didn't reference FlutterCompass in the dispose function. I'm not entirely sure how flutter_compass works, but if it streams data, it might need to be canceled or disposed of in that function as well.
Lastly, make sure that in the final PR, the mock data is either removed or moved to a separate function or utility file.

@3TTemi 3TTemi force-pushed the sd924/rotating-cursor branch from 3af5f4e to 8c5dc29 Compare June 26, 2025 05:14
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.

5 participants