Only show subscribe buttons on actual forum threads

Fixes #118
re621 · Apr 2, 2020 · 16e32e4 · 16e32e4
1 parent 7bf35a8
commit 16e32e4
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/js/components/data/Page.ts b/src/js/components/data/Page.ts
@@ -96,5 +96,6 @@ export const PageDefintion = {
     post: /^\/posts\/\d+\/?$/,
     upload: /\/uploads\/new\/?/,
     forum: /^\/forum_topics\/?.*/,
+    forumPost: /^\/forum_topics\/\d+.*/,
     pool: /^\/pools\/.+/,
 };
diff --git a/src/js/modules/subscriptions/ForumSubscriptions.ts b/src/js/modules/subscriptions/ForumSubscriptions.ts
@@ -1,6 +1,6 @@
 import { UpdateData, UpdateDefinition, SubscriptionSettings, UpdateContent } from "./SubscriptionManager";
 import { Api } from "../../components/api/Api";
-import { Page } from "../../components/data/Page";
+import { Page, PageDefintion } from "../../components/data/Page";
 import { RE6Module, Settings } from "../../components/RE6Module";
 import { Subscription } from "./Subscription";
 import { ApiForumTopic } from "../../components/api/responses/ApiForum";
@@ -36,7 +36,8 @@ export class ForumSubscriptions extends RE6Module implements Subscription {
     }
 
     getButtonElements(): JQuery<HTMLElement> {
-        return $("div#c-forum-topics").first();
+        if (Page.matches(PageDefintion.forumPost)) return $("div#c-forum-topics").first();
+        else return $();
     }
 
     public createSubscribeButton(): JQuery<HTMLElement> {