react / react/react-native

[Android] Impossible to show modal scrim

Đang mở
#39,018 5 bình luận 2 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Component: Modal Issue: Author Provided Repro p: Microsoft Partner Platform: Android
Ngôn ngữ chính
C++
Star
127k
Fork
25.3k
Merge trung bình
1 ngày 23 giờ
Pull request đã merge (30 ngày)
4

Mô tả

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
image image image image

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

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Hướng nghiên cứu

Tái hiện hành vi của Android Modal bằng Snack được liên kết, sau đó kiểm tra Libraries/Modal/Modal.js và ReactAndroid/src/main/java/com/facebook/react/views/modal/ReactModalHostView.java tại các dòng được tham chiếu. Theo dõi nền trắng, FLAG_DIM_BEHIND, updateProperties() và hành vi của mức dim, bao gồm cả commit được tham chiếu. Công việc được xem là hoàn tất khi một Modal trong suốt có thể hiển thị một scrim có thể sử dụng trên toàn bộ màn hình Android với mức dim có thể cấu hình phù hợp.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
android, java, react-native
Lĩnh vực
mobile-dev
Loại issue
Lỗi
Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức độ hoạt động
Ít trao đổi
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
35/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.