[Android] Impossible to show modal scrim
还没有人认领这个 Issue。
- 主要语言
- C++
- 星标
- 127k
- 派生
- 25.3k
- 平均合并
- 1 天 23 小时
- 30 天内合并 PR
- 4
描述
Description
The scrim on the Modal component is impossible to use on Android.
We want to create a simple modal dialog on top of a partially-obscured background (a scrim) -- a common UI pattern.
In theory, should be possible with something like:
<Modal /* snip */>
<Text>This is some text</Text>
<Button title="Close" /* snip */ />
</Modal>
However, this doesn't work. We're given a white backdrop that covers the whole screen, as transparent={false} renders an opaque backdrop.
If we pass transparent={true} to remove the background, the dialog's backdrop is 100% transparent, showing the content behind it.
The next step would be to draw our own scrim in React Native with CSS. However, this results in the scrim not covering the status (notification) bar and the gesture indicator at the bottom. Both of these parts of the screen outside of a typical app's window, and can't be drawn on.
Here's the steps in tabular form:
| Goal | transparent={false} (or undefined) |
transparent={true} |
Custom scrim |
|---|---|---|---|
There are a few issues in the underlying code, which I'll enumerate here.
Forcing white background
The white background is forced by the Modal, on this line of code. It's possible to remove this white background by passing transparent={true}, but that uncovers another bug.
Removing the scrim on transparent dialogs
When the dialog is rendered with transparent={true}, then the modal is rendered without a background. However, when this happens, the FLAG_DIM_BEHIND is cleared from the Dialog, which removes the scrim. This happens on this line of code.
Investigating that uncovers another bug.
The dialog is never updated
When investigating that bug, I discovered that updateProperties() is always short-circuited. Specifically, the window.isActive() check on this line of code is always false.
Removing this fixed the issue. However, it was added in https://github.com/facebook/react-native/commit/7abcaafd6600535825aa8330af7290ba8acea245, which alluded to a crashing issue. I'm not sure whether this specific check was out of paranoia, or what, but it seemed risky to undo without knowing more.
The scrim is hardcoded
Finally, the opacity of the scrim (the "dim amount") is hardcoded on this line of code. It would probably be best if this were left unspecified and/or read from the theme (Theme.FullScreenDialog, I believe) so that apps could customize it if they wish (ie, Theme_backgroundDimAmount or Window_backgroundDimAmount)
React Native Version
0.72.4
Output of npx react-native info
info Fetching system and libraries information...
System:
OS: macOS 13.5
CPU: (10) arm64 Apple M1 Max
Memory: 202.89 MB / 64.00 GB
Shell:
version: "5.9"
path: /bin/zsh
Binaries:
Node:
version: 18.17.1
path: /usr/local/bin/node
Yarn:
version: 1.22.19
path: /opt/homebrew/bin/yarn
npm:
version: 9.6.7
path: /usr/local/bin/npm
Watchman:
version: 2023.08.07.00
path: /opt/homebrew/bin/watchman
Managers:
CocoaPods:
version: 1.12.1
path: /opt/homebrew/bin/pod
SDKs:
iOS SDK:
Platforms:
- DriverKit 22.2
- iOS 16.2
- macOS 13.1
- tvOS 16.1
- watchOS 9.1
Android SDK:
Android NDK: 25.2.9519653
IDEs:
Android Studio: 2022.1 AI-221.6008.13.2211.9619390
Xcode:
version: 14.2/14C18
path: /usr/bin/xcodebuild
Languages:
Java:
version: 17.0.8
path: /opt/homebrew/opt/openjdk@17/bin/javac
Ruby:
version: 2.6.10
path: /usr/bin/ruby
npmPackages:
"@react-native-community/cli": Not Found
react:
installed: 18.2.0
wanted: 18.2.0
react-native: Not Found
react-native-macos: Not Found
npmGlobalPackages:
"react-native": Not Found
Android:
hermesEnabled: Not found
newArchEnabled: Not found
iOS:
hermesEnabled: Not found
newArchEnabled: Not found
Steps to reproduce
Attempt to create the above dialog. I believe that the Snack and the steps above are sufficient for this.
Snack, screenshot, or link to a repository
Snack to demonstrate this: https://snack.expo.dev/@echarlton/modal
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
调研方向
使用链接的 Snack 重现 Android Modal 的行为,然后检查所引用行中的 Libraries/Modal/Modal.js 和 ReactAndroid/src/main/java/com/facebook/react/views/modal/ReactModalHostView.java。跟踪白色背景、FLAG_DIM_BEHIND、updateProperties() 以及 dim amount 的行为,包括所引用的 commit。当透明的 Modal 能够以适当的可配置 dim amount 在整个 Android 显示屏上显示可用的 scrim 时,即表示完成。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- android, java, react-native
- 领域
- mobile-dev
- Issue 类型
- 缺陷
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 活跃度
- 冷清
- 描述清晰度
- 基本清楚
- 新手友好度
- 35/100