BAP modal issues
See original GitHub issueDescription
Found while testing https://github.com/brave/brave-browser/issues/13798
Two issues found with the BAP modal (not tooltip)
- unable to scroll BAP modal when in landscape mode (possibly only show modal in portrait mode --> https://bravesoftware.slack.com/archives/C0816QH3R/p1613161177249100?thread_ts=1613160945.247600&cid=C0816QH3R)
- Styling of the modal looks a bit odd (large white bar at the top which is not in original issue design)
Steps to reproduce
- Set device to Japan/Japanese
- Install
1.22.22
Nightly, enable staging env for Rewards - Relaunch
- Tap on Rewards icon, dismiss onboarding
- Claim UGP grant
- Close app
- Advance date by 3 days
- Orient device to be landscape
- Launch Nightly
- On tooltip, tap Learn More
Actual result
Modal is displayed, can’t scroll
Expected result
Able to scroll in modal (if we keep it in this view). Layout should more closely match original in https://github.com/brave/brave-browser/issues/13798#issue-795249774
Issue reproduces how often
easily
Version/Channel Information:
- Can you reproduce this issue with the current Play Store version? n/a
- Can you reproduce this issue with the current Play Store Beta version? n/a
- Can you reproduce this issue with the current Play Store Nightly version? yes
Device details
- Install type (ARM, x86): ARM
- Device type (Phone, Tablet, Phablet): phone
- Android version: 5.1
Brave version
1.22.22
Website problems only
- Does the issue resolve itself when disabling Brave Shields?
- Does the issue resolve itself when disabling Brave Rewards?
- Is the issue reproducible on the latest version of Chrome?
Additional information
Issue Analytics
- State:
- Created 3 years ago
- Comments:7 (4 by maintainers)
Top Results From Across the Web
Missing translations for BAP deprecation tooltip and modal · Issue ...
Description #14117 fixed translations on desktop. We need the same translations for Android as well. Steps to reproduce Install 1.20.104 in Japanese locale ......
Read more >The difficulty in interpreting gene expression profiling in BAP ...
Conclusions. BAP-negative melanocytic tumors remain a diagnostic dilemma for dermatopathologists. A widely marketed 23-GEP test may not be ...
Read more >BAP complains to PM Imran Khan for not resolving issues
ISLAMABAD: PTI ally BAP has complained to Prime Minister Imran Khan for not resolving Balochistan's problems since he came to power three ...
Read more >(PDF) The Body Action and Posture Coding System (BAP)
We discuss issues for further validation and propose some research applications. KeywordsBody movement–Expression–Coding–Method–Emotion.
Read more >FIRST-ORDER MODAL LOGIC - UCI Canvas
Classically, first-order issues like constant and function symbols, equality, quantification, and definite descriptions, have straightforward formal treat-.
Read more >
Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free
Top Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
@Miyayes @deeppandya assuming we want this in
1.20.x
for the BAP release so I moved it into https://github.com/brave/brave-browser/milestone/173 and labelling it as arelease/blocking
. If we don’t care about getting this into the1.20.x - Release #2
and are fine with this going into1.21.x
, please remove it from the millstone. CCing @bsclifton @rebronVerification passed on Samsung Tab A with Android 10 running 1.21.76 x64 build
https://user-images.githubusercontent.com/17010094/110787931-dc0bca00-8265-11eb-856f-371ab17b5d64.mp4
Verification passed on OnePlus 6T with Android 10 running 1.21.76 x64 build
https://user-images.githubusercontent.com/17010094/110788001-f3e34e00-8265-11eb-84bf-e28da6926e97.mp4