share dialog styling improvements

css/style.css

@@ -155,12 +155,17 @@ div[data-placeholder]:not([data-placeholder=""]):empty::before {
 
 #note-share-options {
     display: none;
+    padding-bottom: 5px;
 }
 
-.selected-share:hover, .unselected-share:hover {
+.selected-share:hover, .unselected-share:hover, .unselected-share span:hover {
     cursor: pointer;
 }
 
+.selected-share, .unselected-share {
+    padding-left: 5px;
+}
+
 /* Modal Content */
 
 .modal-content {

js/script.js

@@ -434,7 +434,8 @@ View.prototype = {
 
                 shareOptions.show();
                 var modalNote = $('.note-active');
-                modalNote.outerHeight(modalNote.outerHeight(true) + shareOptions.outerHeight(true));
+                var startHeight = modalNote.outerHeight(true);
+                modalNote.outerHeight(startHeight + shareOptions.outerHeight(true));
                 sear.on('input', function() {
                     var val = $(this).val().toLowerCase().trim();
                     var lis = neg.children();
@@ -449,6 +450,7 @@ View.prototype = {
                             }
                         }
                     }
+                    modalNote.outerHeight(startHeight + shareOptions.outerHeight(true));
                 });
            });
         });

templates/part.note-modal-editable.php

@@ -10,6 +10,7 @@
 					<button id='cancel-button'><?php p($l->t('Cancel')); ?></button>
 					<button id='save-button'><?php p($l->t('Save')); ?></button>
 				</div>
+				<div style="clear: both;"></div>
 				<div id="note-share-options">
 					<ul id="share-pos">
 					</ul>