From e3ca3c96e5fe87d6ea8123b02ea037cf8255adb9 Mon Sep 17 00:00:00 2001
From: Vladimir Panteleev
Date: Tue, 16 Oct 2018 05:22:29 +0000
Subject: Add "Enable notifications" checkbox in "Add Comment" form
Currently, it is a little to easy to forget to enable notifications
for a package after leaving a comment, thus never being notified of a
reply. Even though the "Enable notifications" link is on the same
page, it is not part of the flow for posting a new comment, and so,
easy to miss.
Most web forums and comment systems include a checkbox to enable
notifications when posting for the first time in a thread. This patch
implements this in aurweb, as well.
Signed-off-by: Eli Schwartz
---
web/html/css/aurweb.css | 5 +++++
web/html/pkgbase.php | 3 +++
web/template/pkg_comment_form.php | 6 ++++++
3 files changed, 14 insertions(+)
(limited to 'web')
diff --git a/web/html/css/aurweb.css b/web/html/css/aurweb.css
index 593c9ae8..ef37bf59 100644
--- a/web/html/css/aurweb.css
+++ b/web/html/css/aurweb.css
@@ -104,6 +104,11 @@
top: 1px;
}
+.comment-enable-notifications {
+ display: inline-block;
+ margin-left: 1em;
+}
+
.rss-icon, .delete-comment, .undelete-comment, .edit-comment, .pin-comment {
-webkit-filter: grayscale(100%);
filter: grayscale(100%);
diff --git a/web/html/pkgbase.php b/web/html/pkgbase.php
index 46ad77e6..b716644c 100644
--- a/web/html/pkgbase.php
+++ b/web/html/pkgbase.php
@@ -120,6 +120,9 @@ if (check_token()) {
} elseif (current_action("do_AddComment")) {
$uid = uid_from_sid($_COOKIE["AURSID"]);
list($ret, $output) = pkgbase_add_comment($base_id, $uid, $_REQUEST['comment']);
+ if ($ret && isset($_REQUEST['enable_notifications'])) {
+ list($ret, $output) = pkgbase_notify(array($base_id));
+ }
$fragment = '#news';
} elseif (current_action("do_EditComment")) {
list($ret, $output) = pkgbase_edit_comment($_REQUEST['comment']);
diff --git a/web/template/pkg_comment_form.php b/web/template/pkg_comment_form.php
index c450c4b3..3feee8fb 100644
--- a/web/template/pkg_comment_form.php
+++ b/web/template/pkg_comment_form.php
@@ -13,6 +13,12 @@