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

add_subscribers_pill: Add button should show error on pending text.

Browse Source 
This commit only changes the behaviour for edit screen of this setting.
We'll tackle the create screens in further commits.
This commit is contained in:
Shubham Padia
2024-12-11 07:42:49 +00:00
committed by Tim Abbott
parent 0ee2543a49
commit f6bedceab3
1 changed files with 12 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
web/src/add_subscribers_pill.ts
View File

@@ -256,14 +256,23 @@ export function set_up_handlers({
}
$parent_container.on("keyup", pill_selector, (e) => {
if (keydown_util.is_enter_event(e)) {
const pill_widget = get_pill_widget();
if (!pill_widget.is_pending() && keydown_util.is_enter_event(e)) {
e.preventDefault();
callback();
}
});
$parent_container.on("click", button_selector, (e) => {
e.preventDefault();
callback();
const pill_widget = get_pill_widget();
if (!pill_widget.is_pending()) {
e.preventDefault();
callback();
} else {
// We are not appending any value here, but instead this is
// a proxy to invoke the error state for a pill widget
// that would usually get triggered on pressing enter.
pill_widget.appendValue(pill_widget.getCurrentText()!);
}
});
}