fixes #4992 (#4994)

* fixes #4992

* make jquery ui a r.js module

* require jqueryui in admin/extend/widgets

public/less/admin/general/navigation.less

@@ -3,6 +3,8 @@
 		width: 100%;
 		min-height: 50px;
 		border: 1px solid #eee;
+		overflow: auto;
+		
 		.active {
 			background-color: #eee;
 		}

public/src/admin/extend/widgets.js

@@ -1,7 +1,7 @@
 "use strict";
 /* global define, app, socket */
 
-define('admin/extend/widgets', function() {
+define('admin/extend/widgets', ['jqueryui'], function(jqueryui) {
 	var Widgets = {};
 
 	Widgets.init = function() {		

public/src/admin/general/navigation.js

@@ -1,7 +1,7 @@
 "use strict";
 /* global define, app, ajaxify, socket, templates */
 
-define('admin/general/navigation', ['translator', 'iconSelect'], function(translator, iconSelect) {
+define('admin/general/navigation', ['translator', 'iconSelect', 'jqueryui'], function(translator, iconSelect, jqueryui) {
 	var navigation = {},
 		available;
 

public/src/admin/modules/selectable.js

@@ -2,14 +2,12 @@
 
 /*globals define*/
 
-define('admin/modules/selectable', function() {
+define('admin/modules/selectable', ['jqueryui'], function(jqueryui) {
 	var selectable = {};
 
 	selectable.enable = function(containerEl, targets) {
-		app.loadJQueryUI(function() {
+		$(containerEl).selectable({
-			$(containerEl).selectable({
+			filter: targets
-				filter: targets
-			});
 		});
 	};
 

src/meta/js.js

@@ -80,7 +80,7 @@ module.exports = function(Meta) {
 			modules: {
 				"Chart.js": './node_modules/chart.js/dist/Chart.min.js',
 				"mousetrap.js": './node_modules/mousetrap/mousetrap.js',
-
+				"jqueryui.js": 'public/vendor/jquery/js/jquery-ui.js',
 				"buzz.js": 'public/vendor/buzz/buzz.js'
 			}
 		}