aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAllan Wang <me@allanwang.ca>2019-09-22 14:42:23 -0700
committerGitHub <noreply@github.com>2019-09-22 14:42:23 -0700
commit8c9d1df82f31b7eca5fd383bf51c8634fa0696ee (patch)
tree08cbb88d343552c3eb9d52b1bf2fd5a2ec6e3617
parent96908453abbd7e40e13af4359d1bce4524b979a6 (diff)
parent8d355d472f579d10707c8a0b65fb424eb283265d (diff)
downloadfrost-8c9d1df82f31b7eca5fd383bf51c8634fa0696ee.tar.gz
frost-8c9d1df82f31b7eca5fd383bf51c8634fa0696ee.tar.bz2
frost-8c9d1df82f31b7eca5fd383bf51c8634fa0696ee.zip
Merge pull request #1552 from AllanWang/notifications-page
Disable click launches from notification page
-rw-r--r--app/src/main/kotlin/com/pitchedapps/frost/web/FrostWebViewClients.kt5
-rw-r--r--app/src/main/res/xml/frost_changelog.xml7
-rw-r--r--app/src/web/ts/click_a.ts5
3 files changed, 15 insertions, 2 deletions
diff --git a/app/src/main/kotlin/com/pitchedapps/frost/web/FrostWebViewClients.kt b/app/src/main/kotlin/com/pitchedapps/frost/web/FrostWebViewClients.kt
index 9e45d4fe..721b2c2d 100644
--- a/app/src/main/kotlin/com/pitchedapps/frost/web/FrostWebViewClients.kt
+++ b/app/src/main/kotlin/com/pitchedapps/frost/web/FrostWebViewClients.kt
@@ -68,6 +68,11 @@ open class FrostWebViewClient(val web: FrostWebView) : BaseWebViewClient() {
private val refresh: SendChannel<Boolean> = web.parent.refreshChannel
private val isMain = web.parent.baseEnum != null
+ override fun doUpdateVisitedHistory(view: WebView, url: String?, isReload: Boolean) {
+ v { "History $url" }
+ super.doUpdateVisitedHistory(view, url, isReload)
+ }
+
protected inline fun v(crossinline message: () -> Any?) = L.v { "web client: ${message()}" }
override fun onPageStarted(view: WebView, url: String?, favicon: Bitmap?) {
diff --git a/app/src/main/res/xml/frost_changelog.xml b/app/src/main/res/xml/frost_changelog.xml
index 3e267f69..f2f8539d 100644
--- a/app/src/main/res/xml/frost_changelog.xml
+++ b/app/src/main/res/xml/frost_changelog.xml
@@ -6,10 +6,15 @@
<item text="" />
-->
+ <version title="v2.4.1" />
+ <item text="Notification tab will keep first page in the same window; fixes marking notifications as read" />
+ <item text="" />
+ <item text="" />
+ <item text="" />
+
<version title="v2.4.0" />
<item text="Removed request services, which potentially caused phishing warnings." />
<item text="Save images with the correct extensions." />
- <item text="" />
<version title="v2.3.2" />
<item text="Disable auto feed refresh by default and add setting to re-enable it" />
diff --git a/app/src/web/ts/click_a.ts b/app/src/web/ts/click_a.ts
index 1fd63683..8a7cc48c 100644
--- a/app/src/web/ts/click_a.ts
+++ b/app/src/web/ts/click_a.ts
@@ -42,7 +42,10 @@
type EventHandler = (e: Event, target: HTMLElement) => Boolean
const _frostGeneral: EventHandler = (e, target) => {
- // Notifications are two layers under
+ // We now disable clicks for the main notification page
+ if (document.getElementById("notifications_list")) {
+ return false
+ }
const url = _parentUrl(target, 2);
return Frost.loadUrl(url);
};