Skip to content
This repository was archived by the owner on Sep 6, 2021. It is now read-only.

Commit 8eb67f4

Browse files
committed
Refactor work set menu sorting event handler
1 parent 5f261f8 commit 8eb67f4

2 files changed

Lines changed: 141 additions & 127 deletions

File tree

src/command/DefaultMenus.js

Lines changed: 3 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@ define(function (require, exports, module) {
3333

3434
var AppInit = require("utils/AppInit"),
3535
Commands = require("command/Commands"),
36+
ContextMenu = require("command/Menus"),
3637
EditorManager = require("editor/EditorManager"),
3738
Menus = require("command/Menus"),
3839
Strings = require("strings");
@@ -280,24 +281,8 @@ define(function (require, exports, module) {
280281
/**
281282
* Dropdown menu for workspace sorting
282283
*/
283-
$("#working-set-option-btn").on("click", function (e) {
284-
var buttonOffset,
285-
buttonHeight;
286-
287-
e.stopPropagation();
288-
289-
if (working_set_settings_cmenu.isOpen()) {
290-
working_set_settings_cmenu.close();
291-
} else {
292-
buttonOffset = $(this).offset();
293-
buttonHeight = $(this).outerHeight();
294-
working_set_settings_cmenu.open({
295-
pageX: buttonOffset.left,
296-
pageY: buttonOffset.top + buttonHeight
297-
});
298-
}
299-
});
300-
284+
Menus.ContextMenu.assignContextMenuToSelector("#working-set-option-btn", working_set_settings_cmenu);
285+
301286
// Prevent the browser context menu since Brackets creates a custom context menu
302287
$(window).contextmenu(function (e) {
303288
e.preventDefault();

0 commit comments

Comments
 (0)