Merge pull request #2775 from netchampfaris/summernote-fixes

summernote fixes

frappe/public/css/desk.css

@@ -662,6 +662,13 @@ fieldset[disabled] .form-control {
   max-height: 300px;
   overflow: auto;
 }
+.modal .note-editor .note-btn-italic,
+.modal .note-editor .note-btn-underline,
+.modal .note-editor [data-original-title="Font Size"],
+.modal .note-editor [data-original-title="Video"],
+.modal .note-editor [data-original-title="Table"] {
+  display: none;
+}
 .search-dialog .modal-dialog {
   width: 768px;
   height: 500px;

frappe/public/js/frappe/form/control.js

@@ -1668,6 +1668,8 @@ frappe.ui.form.ControlTextEditor = frappe.ui.form.ControlCode.extend({
 			}
 		});
 		this.note_editor = $(this.input_area).find('.note-editor');
+		// to fix <p> on enter
+		this.set_input('<div><br></div>');
 	},
 	hide_elements_on_mobile: function() {
 		this.note_editor.find('.note-btn-underline,\

frappe/public/js/frappe/views/communication.js

@@ -530,7 +530,7 @@ frappe.views.CommunicationComposer = Class.extend({
 			var last_email_content = last_email.original_comment || last_email.content;
 
 			fields.content.set_input(
-				'<p><br></p>'
+				'<div><br></div>'
 				+ reply
 				+ "<br><!-- original-reply --><br>"
 				+ '<blockquote>' +
@@ -539,7 +539,7 @@ frappe.views.CommunicationComposer = Class.extend({
 					last_email_content +
 				'<blockquote>');
 		} else {
-			fields.content.set_input(reply);
+			fields.content.set_input("<div><br></div>" + reply);
 		}
 	},
 	setup_awesomplete: function() {

frappe/public/less/desk.less

@@ -531,6 +531,17 @@ textarea.form-control {
 	}
 }
 
+// hide some buttons in modal
+.modal .note-editor {
+	.note-btn-italic,
+	.note-btn-underline,
+	[data-original-title="Font Size"],
+	[data-original-title="Video"],
+	[data-original-title="Table"] {
+		display: none;
+	}
+}
+
 .search-dialog {
 	.modal-dialog {
 		width: 768px;