notifications: Avoid misleading href on scroll to bottom notice.

Ykumar1415 · Feb 3, 2023 · a8ce7df · a8ce7df
1 parent dd95f79
commit a8ce7df
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/static/js/notifications.js b/static/js/notifications.js
@@ -612,7 +612,6 @@ export function notify_local_mixes(messages, need_user_to_scroll) {
 
         let reason = get_local_notify_mix_reason(message);
 
-        const above_composebox_narrow_url = get_above_composebox_narrow_url(message);
         const link_msg_id = message.id;
 
         if (!reason) {
@@ -622,7 +621,8 @@ export function notify_local_mixes(messages, need_user_to_scroll) {
                 notify_above_composebox(
                     reason,
                     "compose_notification_scroll_to_message",
-                    above_composebox_narrow_url,
+                    // Don't display a URL on hover for the "Scroll to bottom" link.
+                    null,
                     link_msg_id,
                     link_text,
                 );
@@ -634,6 +634,7 @@ export function notify_local_mixes(messages, need_user_to_scroll) {
             continue;
         }
 
+        const above_composebox_narrow_url = get_above_composebox_narrow_url(message);
         const link_class = "compose_notification_narrow_by_topic";
         const link_text = $t(
             {defaultMessage: "Narrow to {message_recipient}"},

diff --git a/static/templates/compose_banner/message_sent_banner.hbs b/static/templates/compose_banner/message_sent_banner.hbs
@@ -1,4 +1,4 @@
 <div class="compose-notifications-content">
-    {{note}} {{#if link_class}}<a href="{{above_composebox_narrow_url}}" class="{{link_class}}" data-message-id="{{link_msg_id}}">{{link_text}}</a>{{/if}}
+    {{note}} {{#if link_class}}<a {{#if above_composebox_narrow_url}}href="{{above_composebox_narrow_url}}"{{/if}} class="{{link_class}}" data-message-id="{{link_msg_id}}">{{link_text}}</a>{{/if}}
     <button type="button" class="out-of-view-notification-close close">&times;</button>
 </div>