paulmataruso/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2025-10-23 04:52:12 +00:00
Code Issues Packages Projects Releases Wiki Activity

notifications: Avoid misleading href on scroll to bottom notice.

Browse Source
This commit is contained in:
Tim Abbott
2023-02-03 15:19:08 -08:00
parent dd95f79ab7
commit a8ce7df649
2 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
static/js/notifications.js
View File

@@ -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}"},

2
static/templates/compose_banner/message_sent_banner.hbs
View File

@@ -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>