-
Notifications
You must be signed in to change notification settings - Fork 511
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
[XM] Fix dontlink by fixing some NSAccessibility binding mistakes #143
Conversation
@@ -22049,7 +22049,7 @@ partial interface NSAccessibilityAttributes { | |||
[Field ("NSAccessibilityDisclosedRowsAttribute")] | |||
NSString DisclosedRowsAttribute { get; } | |||
|
|||
[Field ("NSAccessibilityDisclosedByRowsAttribute")] | |||
[Field ("NSAccessibilityDisclosedByRowAttribute")] | |||
NSString DisclosedByRowsAttribute { get; } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Managed name should be fixed too (extra s
)
IIRC you can just create an interface without any attribute and have it inlined (or I dreamt that feature). |
@spouliot I could have sworn I tried that, but testing suggests I didn't. Let me rework... |
Build success |
LGTM if dontlink-mac is happy :) |
Build success |
1 similar comment
Build success |
LGTM. |
This commit was not run against dontlink and sadness occurred: f535df6
This fixes things up.
I did not want to expose the previously named NSObjectAccessibilityExtensions on NSObject, even if that's what the header says, since that will really dirty up auto complete. Oh and:
BOOL oDoes = [o respondsToSelector:@selector(accessibilityHitTest:)]; // NSObject * o - NO
BOOL vDoes = [v respondsToSelector:@selector(accessibilityHitTest:)]; // NSView * v - YES
So I did a copy/paste. Which is terrible. Not sure a better way (open to suggestions).