Open ErikBjare opened 3 years ago
This pull request introduces 1 alert when merging 107c00d796101463d5a01ede7b1077b6890d4ab8 into 4837cc0540102ac8354e8f0826cf3363dfcbc41f - view on LGTM.com
new alerts:
This pull request introduces 1 alert and fixes 2 when merging 20a2ec52de4ea780fab841754e5b32facd3b11a1 into 4837cc0540102ac8354e8f0826cf3363dfcbc41f - view on LGTM.com
new alerts:
fixed alerts:
Just wanted to see how easy it was, and it was pretty easy (but there are problems).
Not sure if we should merge something like this quite yet, but I think people want something like it. Would make it feel more like a native app.
An issue is that the settings don't cross between browser and webview, which makes for a confusing UX despite the warning.