Skip to content
This repository has been archived by the owner on Aug 24, 2022. It is now read-only.

Update crate #36

Merged
merged 10 commits into from
Mar 20, 2022
Merged

Update crate #36

merged 10 commits into from
Mar 20, 2022

Conversation

riverar
Copy link
Collaborator

@riverar riverar commented Feb 26, 2022

Adds compatibility for windows (0.34), samples, and interop helpers to unwrap Microsoft.UI types for GDI compatibility.

Fixes #34, fixes #35

@riverar riverar marked this pull request as ready for review February 27, 2022 00:58
@kennykerr
Copy link
Collaborator

Cargo.lock should be gitignore'd for library crates.

Cargo.toml Outdated Show resolved Hide resolved
@kennykerr
Copy link
Collaborator

pub mod interop;

Not sure we need a nested module for this. Can we just put it all in the windows_app::core module?


Refers to: src/core/mod.rs:1 in fb04deb. [](commit_id = fb04deb, deletion_comment = False)

@kennykerr
Copy link
Collaborator

    HWND(window_id.Value.try_into().unwrap())

You really should be calling Windowing_GetWindowFromWindowId. 😎


Refers to: src/core/interop/windowing/mod.rs:20 in fb04deb. [](commit_id = fb04deb, deletion_comment = False)


fn main() {
let start = std::time::Instant::now();
let mut output_path = std::path::PathBuf::from(windows_metadata::workspace_dir());
Copy link
Collaborator

Choose a reason for hiding this comment

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

The workspace_dir helper is going away - just use a relative path here.

kennykerr
kennykerr previously approved these changes Mar 1, 2022
@riverar
Copy link
Collaborator Author

riverar commented Mar 2, 2022

Going to pause ⏸️ merging this until we can chat next week about some UX concerns.

@riverar riverar merged commit bed47ee into master Mar 20, 2022
@riverar riverar deleted the rafael/0.3 branch March 20, 2022 21:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unidiomatic use of Result::and_then in README Feature "UI_Xaml" doesn't compile with windows-rs 0.29-git
2 participants