CodeCow/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2026-07-09 21:21:47 +08:00
Code Issues Actions 3 Packages Projects Releases Wiki Activity

tooltip: Remove dead code related to old tooltips system.
Some checks failed
Code scanning / CodeQL (push) Has been cancelled
Details
Zulip production suite / Ubuntu 22.04 production build (push) Has been cancelled
Details
Zulip CI / ${{ matrix.name }} (zulip/ci:bookworm, true, false, Debian 12 (Python 3.11, backend + documentation), bookworm) (push) Has been cancelled
Details
Zulip CI / ${{ matrix.name }} (zulip/ci:jammy, false, true, Ubuntu 22.04 (Python 3.10, backend + frontend), jammy) (push) Has been cancelled
Details
Zulip CI / ${{ matrix.name }} (zulip/ci:noble, false, false, Ubuntu 24.04 (Python 3.12, backend), noble) (push) Has been cancelled
Details
Zulip production suite / ${{ matrix.name }} (zulip/ci:bookworm, --test-custom-db, Debian 12 production install with custom db name and user, bookworm) (push) Has been cancelled
Details
Zulip production suite / ${{ matrix.name }} (zulip/ci:jammy, , Ubuntu 22.04 production install and PostgreSQL upgrade with pgroonga, jammy) (push) Has been cancelled
Details
Zulip production suite / ${{ matrix.name }} (zulip/ci:noble, , Ubuntu 24.04 production install, noble) (push) Has been cancelled
Details
Zulip production suite / ${{ matrix.name }} (zulip/ci:bookworm-7.0, 7.0 Version Upgrade, bookworm) (push) Has been cancelled
Details
Zulip production suite / ${{ matrix.name }} (zulip/ci:bookworm-8.0, 8.0 Version Upgrade, bookworm) (push) Has been cancelled
Details
Zulip production suite / ${{ matrix.name }} (zulip/ci:jammy-6.0, 6.0 Version Upgrade, jammy) (push) Has been cancelled
Details
Zulip production suite / ${{ matrix.name }} (zulip/ci:noble-9.0, 9.0 Version Upgrade, noble) (push) Has been cancelled
Details

Browse Source
This commit is contained in:
Aman Agrawal 2025-03-03 05:23:27 +00:00 committed by Tim Abbott
parent 7ae1a2c480
commit bb4d77b9f5
5 changed files with 0 additions and 62 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
web/src/click_handlers.ts
View File

@ -259,7 +259,6 @@ export function initialize(): void {
const local_id = $(this).attr("data-reaction-id")!;
const message_id = rows.get_message_id(this);
reactions.process_reaction_click(message_id, local_id);
$(".tooltip").remove();
});
$("body").on("click", ".reveal_hidden_message", (e) => {
@ -465,7 +464,6 @@ export function initialize(): void {
e.preventDefault();
e.stopPropagation();
sidebar_ui.hide_userlist_sidebar();
$(".tooltip").remove();
});
// Doesn't show tooltip on touch devices.

4
web/src/message_view.ts
View File

@ -613,10 +613,6 @@ export let show = (raw_terms: NarrowTerm[], show_opts: ShowMessageViewOpts): voi
inbox_ui.hide();
}
// Open tooltips are only interesting for current narrow,
// so hide them when activating a new one.
$(".tooltip").hide();
blueslip.debug("Narrowed", {
operators: terms.map((e) => e.operator),
trigger: opts ? opts.trigger : undefined,

4
web/styles/left_sidebar.css
View File

@ -50,10 +50,6 @@
margin-right: 10px;
}
.tooltip {
max-width: 18em;
}
.masked_unread_count {
/* 8px at 16px/14em */
font-size: 0.5em;

10
web/styles/portico/stats.css
View File

@ -121,16 +121,6 @@ p {
text-decoration: none;
}
.tooltip-inner {
padding: 10px;
max-width: 350px;
font-size: 0.75rem;
font-weight: 400;
text-align: left;
line-height: 1.4;
}
.last-update {
margin: 0 0 30px;

42
web/styles/zulip.css
View File

@ -702,42 +702,6 @@ strong {
white-space: nowrap;
}
.tooltip {
&.in {
font-size: 12px;
line-height: 1.3;
opacity: 1;
&.left {
margin-left: -12px;
margin-top: 3px;
}
}
.tooltip-inner {
background-color: hsl(0deg 0% 7% / 80%);
padding: 3px 5px;
}
/*
Since hover and click are activated together on touchscreen
devices, the hover state persists until the next click, creating
awkward UI where the tooltip sticks around forever :(.
To resolve this, we just hide the tooltip for touch-events on
touch-enabled devices resolving the above problem. This means
that tooltips will never appear on touchscreen devices, but that's
probably a reasonable tradeoff here.
Source: https://drafts.csswg.org/mediaqueries-4/#mf-interaction
*/
@media (hover: none) {
visibility: hidden !important;
}
}
.buddy_list_tooltip_content {
text-align: left;
word-wrap: break-word;
@ -1995,12 +1959,6 @@ body:not(.hide-left-sidebar) {
}
}
.tooltip_right_arrow {
position: relative;
top: -1px;
font-weight: 600;
}
.spectator_login_for_image_button {
max-width: 250px;
height: 50px;