-
Notifications
You must be signed in to change notification settings - Fork 983
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
fix: white panel when the bottom sheet is launched #20167
Conversation
Jenkins BuildsClick to see older builds (24)
|
:overlay {:interceptTouchOutside true | ||
:handleKeyboardEvents true}} | ||
opts)}}))) | ||
(let [theme (:theme opts) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thank you @jo-mut i've looked into show-overlay usages and I didn't find where we have :theme in opts, so probably it's better to remove it from code
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it seems like (:theme opts)
here (options/statusbar-and-navbar-options (:theme opts) nil nil)
is always nil
this should be fixed I guess
we should move (rf/reg-fx :show-toasts
(fn [[view-id theme]]
(let [[rnn-status-bar nav-bar-color] (get-status-nav-color view-id theme)]
(show-overlay "toasts"
(assoc (options/statusbar-and-navbar-options nil rnn-status-bar nav-bar-color) |
a943dc4
to
7d40f81
Compare
Screen_Recording_20240524_143631_Status_Debug.mp4 |
9d95478
to
5dca94d
Compare
:overlay {:interceptTouchOutside true | ||
:handleKeyboardEvents true}} | ||
opts)}}))) | ||
(let [theme (rf/sub [:theme])] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's illegal, (rf/sub [:theme])
can't be used here , subscription can be used only in views, so you have to pass theme as parameter
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
still not fixed properly
@flexsurfer I had missed your comment above am fixing that now |
(show-overlay "toasts" | ||
(assoc (options/statusbar-and-navbar-options nil rnn-status-bar nav-bar-color) | ||
:overlay | ||
{:interceptTouchOutside false}))))) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also this should keep {:interceptTouchOutside false}
here
this is fixed here #20258 (comment). |
fixes #20114
Summary
The white panel appears when the user initiates any action that launches the bottom sheet. This component does not have a theme property when I inspected it using re-frisk but we do have the global theme value of the app.
In this pr we get this global value of the applied theme and use that to determine which background needs to be applied on the navigation bar.
The issue is resolved
Screen_Recording_20240524_101819_Status_Debug.mp4
status: ready