paulmataruso/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2025-11-05 06:23:38 +00:00
Code Issues Packages Projects Releases Wiki Activity

compose: Fix compose fade not updating on deleting user pill.

Browse Source 
This commit makes the compose.update_fade as an exported
function and adds the listener to update the fade manually
as deleting the pill was not calling this functions.

Fixes #18865
This commit is contained in:
Priyam Seth
2021-08-06 22:37:31 +05:30
committed by Tim Abbott
parent da815db372
commit 5a4318d046
2 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
static/js/compose.js
View File

@@ -89,7 +89,7 @@ export function clear_preview_area() {
$("#compose .markdown_preview").show(); $("#compose .markdown_preview").show();
} }
function update_fade() { export function update_fade() {
if (!compose_state.composing()) { if (!compose_state.composing()) {
return; return;
} }

3
static/js/input_pill.js
View File

@@ -3,6 +3,7 @@ import $ from "jquery";
import render_input_pill from "../templates/input_pill.hbs"; import render_input_pill from "../templates/input_pill.hbs";
import * as blueslip from "./blueslip"; import * as blueslip from "./blueslip";
import * as compose from "./compose";
import * as ui_util from "./ui_util"; import * as ui_util from "./ui_util";
// See https://zulip.readthedocs.io/en/latest/subsystems/input-pills.html // See https://zulip.readthedocs.io/en/latest/subsystems/input-pills.html
@@ -355,6 +356,8 @@ export function create(opts) {
funcs.removePill(id); funcs.removePill(id);
$next.trigger("focus"); $next.trigger("focus");
compose.update_fade();
}); });
store.$parent.on("click", function (e) { store.$parent.on("click", function (e) {