Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@chinmaygarde
Copy link
Member

Adds all the appropriate runtime version checks. I am not able to test the stylus altitude and azimuth readings because I don't have one (can we get one please). I have tried to match documented behavior though.

// Discussion:
// Sweep direction is the same. Phase of M_PI_2.
pointer_data.orientation = [touch azimuthAngleInView:nil] - M_PI_2;
}
Copy link
Contributor

Choose a reason for hiding this comment

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

+1 for detailed comments!

@jason-simmons
Copy link
Member

LGTM

The license file needs to be updated for platform_version

Copy link
Member

@xster xster left a comment

Choose a reason for hiding this comment

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

LG


// pressure_min is always 0.0
if (_platformSupportsTouchPressure) {
// These properties were introduced in iOS 9.0.
Copy link
Member

Choose a reason for hiding this comment

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

should we add these version limitations to pointer_data.h or pointer.dart?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants