-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Update the Default Browser Modal #12390
Comments
@AlexeyBarabash i will start looking into this issue. if you haven't started working on it. |
@karenkliu @jamesmudgett UI in 2nd and 3rd are handled by the android system and the UI depends on android OS on the device. |
@deeppandya I thought we were able to at least tweak the text a little though? |
@karenkliu to keep the consistent, it doesn't allow to change anything on the dialog. i can try with custom UI but i think we should go with default android UI. |
@karenkliu i am updating logic to show the modal based on below, |
one quick question : should we still keep the existing Toast and notification we have on android for default browser ? |
☝️ Yes |
Logic update : |
Verification passed on OnePlus 6T running 1.20.98 x64 build
Verification passed on Samsung Tab A running 1.20.98 x64 build
Logged follow-up issue(#13958) for missing translations |
@anthonypkeane The layout for set Brave as default on Android tablets looks a bit funky; is it worth fixing in a followup? |
@karenkliu @anthonypkeane i think we should start having spec for tablet if the UI is important for tablet users. because we mainly focus on phone UI so some changes would look weird for huge screens and for tablet UI, we would have to explicitly make changes in UI for tablet version. So if we different spec for both, it would be ideal. |
Description
Please update the Default Browser call out on Android as per the design below.
Designs
Toast message:
![image](https://user-images.githubusercontent.com/2388823/105763304-7dcf9580-5f0a-11eb-88e3-a035e70374ec.png)
Select Brave Browser and tap ‘Always’
Assets
Figma: https://www.figma.com/file/atoL4LN2kd8ILnPGwohUoV/?node-id=1369%3A18913
The text was updated successfully, but these errors were encountered: