-
Notifications
You must be signed in to change notification settings - Fork 1.6k
feat!: support previous non-E2E #1817
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
base: master
Are you sure you want to change the base?
feat!: support previous non-E2E #1817
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #1817 +/- ##
==========================================
+ Coverage 72.24% 72.75% +0.51%
==========================================
Files 23 23
Lines 1841 1828 -13
==========================================
Hits 1330 1330
+ Misses 511 498 -13 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
You might be able to use |
b4e4b54
to
250a0c6
Compare
250a0c6
to
7859c97
Compare
56fff49
to
7184110
Compare
…n Android API 35 Overriding Cordova's default themes for API 35 NB: It looks like better E2E support is coming to Cordova Android 15 - see apache/cordova-android#1817
Last call for any additional reviews. Will be merged in tomorrow. |
Is this PR intended to make the current statusbar plug-in deprecated in the future? We currently use the config StatusBarOverlaysWebView from that plugin. If the plug-in is to be deprecated, will you include this option in each platform? |
2fbdc58
to
bfa1941
Compare
Motivation and Context
Description
StatusBar JS API:
window.statusbar.visible = <true|false>
window.statusbar.setBackgroundColor('#AARRGGBB')
`config.xml preferences:
This PR reuses all of the previous preferences. Order of imports listed below
StatusBarBackgroundColor
BackgroundColor
BackgroundColor
Testing
npm t
Checklist
(platform)
if this change only applies to one platform (e.g.(android)
)