-
-
Notifications
You must be signed in to change notification settings - Fork 175
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
fix: #268 - Add API for MacOS NSWindow::isDocumentEdited() #287
Conversation
Thanks for contribution. It seems the PR you linked, just got merged, I guess we need to wait for release only. |
c65d01d
to
2b50be3
Compare
Done. I'm not clear on the difference between |
This fixes tauri-apps#268 and therefore tauri-apps/tauri#3125 The setter API is added upstream in the cocoa crate (servo/core-foundation-rs PR tauri-apps#493)
2b50be3
to
1ff2ec0
Compare
This PR needs to wait for a new release of |
src/platform_impl/macos/window.rs
Outdated
unsafe { | ||
self | ||
.ns_window | ||
.isDocumentEdited() |
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.
looks like a new version of cocoa won't be released anytime soon and we have been waiting for ages, can you just inline the objc call from this PR https://github.com/servo/core-foundation-rs/pull/493/files here
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.
Thank you!
* Add API for MacOS NSWindow::isDocumentEdited() This fixes #268 and therefore tauri-apps/tauri#3125 The setter API is added upstream in the cocoa crate (servo/core-foundation-rs PR #493) * use objc directly * fmt * cast to i8 * Update src/platform/macos.rs
lol the PR is so old, it was targeting |
What kind of change does this PR introduce?
This fixes #268 and therefore tauri-apps/tauri#3125
isDocumentEdited
#268Does this PR introduce a breaking change?
Checklist
fix #___, #___
)Other information
The setter API is added upstream in the cocoa crate (servo/core-foundation-rs PR #493)), which will need to be merged & released before this PR.