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

Initial Android support #86

Merged
merged 7 commits into from
Jul 1, 2024
Merged

Initial Android support #86

merged 7 commits into from
Jul 1, 2024

Conversation

matthunz
Copy link
Contributor

@matthunz matthunz commented Jun 25, 2024

Provides the initial working support for Android. Interactivity is blocked on touch and gesture support, but HTML can be displayed and seems to work across app launches.

This also adds docsrs configuration to provide information about android-only functions

Copy link
Collaborator

@nicoburns nicoburns left a comment

Choose a reason for hiding this comment

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

Mostly looks good to me. I wonder whether we need to gate the menu stuff on "not android platform" too though? Otherwise you'd have to build with different feature flags on different platforms. Maybe that's fine?

let proxy = event_loop.create_proxy();

// Multiwindow ftw
let mut windows: HashMap<WindowId, window::View<'_, Doc>> = HashMap::new();
let mut pending_windows = Vec::new();

pending_windows.push(window);
let menu_channel = MenuEvent::receiver();

#[cfg(feature = "muda")]
Copy link
Collaborator

Choose a reason for hiding this comment

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

I reckon this feature should probably be called "menu" "menubar".

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Totally agree 👍 I went with the name menu in 823f309 but definitely open to menubar as well

I also made the feature flag default, but not usable from Android/IOS, which I think could be easier for end users

@jkelleyrtp jkelleyrtp merged commit f7fe94f into DioxusLabs:main Jul 1, 2024
8 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