Skip to content

[ui][android] DO_NOT_MERGE Add native drawer.#41384

Draft
aleqsio wants to merge 2 commits intomainfrom
@aleqsio/12-03-_ui_android_do_not_merge_add_native_drawer
Draft

[ui][android] DO_NOT_MERGE Add native drawer.#41384
aleqsio wants to merge 2 commits intomainfrom
@aleqsio/12-03-_ui_android_do_not_merge_add_native_drawer

Conversation

@aleqsio
Copy link
Copy Markdown
Contributor

@aleqsio aleqsio commented Dec 3, 2025

Why

I was working on the wikireader app, and the main feature I was missing was a navigation drawer.

It's still a draft, need to think how and where to land this but this aligns with expo-router APIs.

image.png

How

Copied over @react-navigation/drawer and changed code to use the new expo-ui NavigationDrawer component.

This also allows using compose components as items of the drawer, though the API is restrictive enough where it's hard to fully customize it I think.

Will explore using NavigationRail as an option/instead of the drawer.

Test Plan

Checklist

Copy link
Copy Markdown
Contributor Author

aleqsio commented Dec 3, 2025

@expo-bot
Copy link
Copy Markdown
Collaborator

expo-bot commented Dec 3, 2025

Hi there! 👋 I'm a bot whose goal is to ensure your contributions meet our guidelines.

I've found some issues in your pull request that should be addressed (click on them for more details) 👇

⚠️ Suggestion: Missing changelog entries


Your changes should be noted in the changelog, e.g.:
- [android] DO_NOT_MERGE Add native drawer. ([#41384](https://github.com/expo/expo/pull/41384) by [@aleqsio](https://github.com/aleqsio))
Read Updating Changelogs guide and consider adding an appropriate entry to the following changelogs:


Generated by ExpoBot 🤖 against fcdf8f9

@expo-bot expo-bot added the bot: suggestions ExpoBot has some suggestions label Dec 3, 2025
@expo-bot
Copy link
Copy Markdown
Collaborator

expo-bot commented Dec 3, 2025

The Pull Request introduced fingerprint changes against the base commit: 5a2e7f8

Fingerprint diff
[
  {
    "op": "changed",
    "beforeSource": {
      "type": "dir",
      "filePath": "../../packages/expo-modules-core",
      "reasons": [
        "expoAutolinkingIos",
        "expoAutolinkingAndroid",
        "expoAutolinkingIos"
      ],
      "hash": "0ea8b2ac87a722a59e9bafc5ce85bd9e0859525a"
    },
    "afterSource": {
      "type": "dir",
      "filePath": "../../packages/expo-modules-core",
      "reasons": [
        "expoAutolinkingIos",
        "expoAutolinkingAndroid",
        "expoAutolinkingIos"
      ],
      "hash": "245aae8585c97035fe6f85036685f63c340b2193"
    }
  },
  {
    "op": "changed",
    "beforeSource": {
      "type": "dir",
      "filePath": "../../packages/expo-ui/android",
      "reasons": [
        "expoAutolinkingAndroid"
      ],
      "hash": "04adccbab30396483749b362cdbc3024f7977043"
    },
    "afterSource": {
      "type": "dir",
      "filePath": "../../packages/expo-ui/android",
      "reasons": [
        "expoAutolinkingAndroid"
      ],
      "hash": "f5ef1c2a947d053b6dbe18094796ab0ad7b641c3"
    }
  }
]

Generated by PR labeler 🤖

Base automatically changed from @aleqsio/12-03-_ui_android_fix_events_for_functional_view_definitions to main December 16, 2025 22:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants