Skip to content
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

gdk4-macos: manually implement native_window method #1938

Merged
merged 2 commits into from
Dec 18, 2024

Conversation

nacho
Copy link
Contributor

@nacho nacho commented Dec 9, 2024

For now we need to depend on the cocoa crate in order to implement this method since it is the only decent crate implementing the cocoa api.

@nacho nacho requested a review from bilelmoussaoui as a code owner December 9, 2024 11:29
gdk4-macos/src/auto/macos_surface.rs Outdated Show resolved Hide resolved
gdk4-macos/Cargo.toml Outdated Show resolved Hide resolved
@nacho nacho force-pushed the wip/nacho/nswindow branch from be2391a to bf76556 Compare December 9, 2024 11:43
@nacho nacho force-pushed the wip/nacho/nswindow branch 2 times, most recently from 8ab2721 to 7a38527 Compare December 9, 2024 11:51
gdk4-macos/Gir.toml Outdated Show resolved Hide resolved
@nacho nacho force-pushed the wip/nacho/nswindow branch 4 times, most recently from 27885fa to db6353d Compare December 9, 2024 13:02
@nacho nacho force-pushed the wip/nacho/nswindow branch 3 times, most recently from 59d6bde to 37c95b3 Compare December 9, 2024 17:04
@bilelmoussaoui
Copy link
Member

The changes looks good to me, but the docs CI as being run on a linux machine seems to fail at linking cocoa, which is weird as it is an optional feature...

@nacho
Copy link
Contributor Author

nacho commented Dec 9, 2024

The changes looks good to me, but the docs CI as being run on a linux machine seems to fail at linking cocoa, which is weird as it is an optional feature...

I wonder if it has something to do with the fact that the feature name is cocoa and the dependency is also cocoa...

@sdroege
Copy link
Member

sdroege commented Dec 10, 2024

The docs build uses --all-features, that's why

@nacho
Copy link
Contributor Author

nacho commented Dec 10, 2024

I wonder how this works for windows then... checking

@nacho nacho force-pushed the wip/nacho/nswindow branch from 6e08ebf to 131dd95 Compare December 10, 2024 09:23
@bilelmoussaoui
Copy link
Member

Can you rebase this one?

For now we need to depend on the cocoa crate in order to implement
this method since it is the only decent crate implementing
the cocoa api.
@nacho nacho force-pushed the wip/nacho/nswindow branch from 131dd95 to 0dde555 Compare December 18, 2024 09:00
@bilelmoussaoui bilelmoussaoui merged commit d3c51de into gtk-rs:main Dec 18, 2024
40 checks passed
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.

3 participants