Merge pull request #1458 from anandpdoshi/minor/comment-on-ctrl-enter
[minor] post comment on cmd+enter or ctrl+enter
This commit is contained in:
commit
d1aa3bb2f4
2 changed files with 11 additions and 3 deletions
|
|
@ -46,7 +46,7 @@ frappe.desk.pages.Messages = Class.extend({
|
|||
if(comment.modified_by !== user) {
|
||||
frappe.utils.notify(__("Message from {0}", [comment.comment_by_fullname]), comment.comment);
|
||||
}
|
||||
if (frappe.get_route()[0] === 'messages') {
|
||||
if (frappe.get_route()[0] === 'messages' && comment.owner !== user) {
|
||||
var current_contact = $(cur_page.page).find('[data-contact]').data('contact');
|
||||
var on_broadcast_page = current_contact === user;
|
||||
if ((current_contact == comment.owner) || (on_broadcast_page && comment.broadcast)) {
|
||||
|
|
@ -101,7 +101,7 @@ frappe.desk.pages.Messages = Class.extend({
|
|||
|
||||
this.page.main.html($(frappe.render_template("messages_main", { "contact": contact })));
|
||||
|
||||
this.page.main.find(".messages-textarea").on("focusout", function() {
|
||||
var text_area = this.page.main.find(".messages-textarea").on("focusout", function() {
|
||||
// on touchscreen devices, scroll to top
|
||||
// so that static navbar and page head don't overlap the textarea
|
||||
if (frappe.dom.is_touchscreen()) {
|
||||
|
|
@ -109,7 +109,7 @@ frappe.desk.pages.Messages = Class.extend({
|
|||
}
|
||||
});
|
||||
|
||||
this.page.main.find(".btn-post").on("click", function() {
|
||||
var post_btn = this.page.main.find(".btn-post").on("click", function() {
|
||||
var btn = $(this);
|
||||
var message_box = btn.parents(".message-box");
|
||||
var textarea = message_box.find("textarea");
|
||||
|
|
@ -138,6 +138,10 @@ frappe.desk.pages.Messages = Class.extend({
|
|||
}
|
||||
});
|
||||
|
||||
text_area.keydown("meta+return ctrl+return", function(e) {
|
||||
post_btn.trigger("click");
|
||||
});
|
||||
|
||||
this.page.wrapper.find(".page-head .message-to").html(frappe.user.full_name(contact));
|
||||
|
||||
this.make_message_list(contact);
|
||||
|
|
|
|||
|
|
@ -28,6 +28,10 @@ frappe.ui.form.Comments = Class.extend({
|
|||
}
|
||||
});
|
||||
|
||||
this.input.keydown("meta+return ctrl+return", function(e) {
|
||||
me.button.trigger("click");
|
||||
});
|
||||
|
||||
this.email_check = this.wrapper.find(".timeline-head input[type='checkbox']")
|
||||
.on("change", function() {
|
||||
me.button.html($(this).prop("checked") ? __("Compose") : __("Comment"));
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue