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

move_modal: Focus input in "Move messages" and "Move topic" modals.

Browse Source 
If user has permissions to move the message(s) to a different stream
and can't edit topic name then focus stream input; else, focus topic
input on "Move messages" and "Move topic" modals render by
`ui_util.place_caret_at_end($(".move_messages_edit_topic")[0])` to
position the cursor at end on focus.

Wrapped if else logic in function focus_on_move_modal_render()
and passed it to on_shown parameter of dialog_widget.launch()
to focus stream/topic input once widget is rendered.

Fixes #24805
This commit is contained in:
Hardik Dharmani
2023-04-18 21:09:16 +05:30
committed by Tim Abbott
parent cfbcbe901a
commit bb78d9741f
1 changed files with 10 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
web/src/stream_popover.js
View File

@@ -22,6 +22,7 @@ import * as stream_data from "./stream_data";
import * as stream_settings_ui from "./stream_settings_ui";
import * as sub_store from "./sub_store";
import * as ui_report from "./ui_report";
import * as ui_util from "./ui_util";
import * as unread_ops from "./unread_ops";
// We handle stream popovers and topic popovers in this
// module. Both are popped up from the left sidebar.
@@ -381,6 +382,14 @@ export function build_move_topic_to_stream_popover(
});
}
function focus_on_move_modal_render() {
if (!disable_stream_input && args.disable_topic_input) {
$("#select_stream_widget .button").trigger("focus");
} else {
ui_util.place_caret_at_end($(".move_messages_edit_topic")[0]);
}
}
dialog_widget.launch({
html_heading: modal_heading,
html_body: render_move_topic_to_stream(args),
@@ -388,6 +397,7 @@ export function build_move_topic_to_stream_popover(
id: "move_topic_modal",
on_click: move_topic,
loading_spinner: true,
on_shown: focus_on_move_modal_render,
post_render: move_topic_post_render,
});
}