adobe/brackets

Function _keydownHook has 75 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    InlineMenu.prototype._keydownHook = function (event) {
        var keyCode,
            self = this;

        // positive distance rotates down; negative distance rotates up

Function _buildListView has 39 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    InlineMenu.prototype._buildListView = function (items) {
        var self            = this,
            view            = { items: [] },
            _addItem;

Avoid too many return statements within this function.

Open

            return true;

Avoid too many return statements within this function.

Open

        return false;

Similar blocks of code found in 2 locations. Consider refactoring.

Open

            if (self.selectedIndex < 0) {
                // set the initial selection
                pos = (distance > 0) ? distance - 1 : len - 1;

            } else {

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    InlineMenu.prototype.open = function (items) {
        Menus.closeAll();

        this._buildListView(items);

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        function _itemsPerPage() {
            var itemsPerPage = 1,
                $items = self.$menu.find("li.inlinemenu-item"),
                $view = self.$menu.find("ul.dropdown-menu"),
                itemHeight;

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    InlineMenu.prototype.close = function () {
        this.opened = false;

        if (this.$menu) {
            this.$menu.removeClass("open");

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    InlineMenu.prototype.isOpen = function () {
        // We don't get a notification when the dropdown closes. The best
        // we can do is keep an "opened" flag and check to see if we
        // still have the "open" class applied.
        if (this.opened && !this.$menu.hasClass("open")) {

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

            } else if (event.shiftKey &&
                    (event.keyCode === KeyEvent.DOM_VK_UP ||
                     event.keyCode === KeyEvent.DOM_VK_DOWN ||
                     event.keyCode === KeyEvent.DOM_VK_PAGE_UP ||
                     event.keyCode === KeyEvent.DOM_VK_PAGE_DOWN)) {

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        return (keyCode === KeyEvent.DOM_VK_UP || keyCode === KeyEvent.DOM_VK_DOWN ||
            keyCode === KeyEvent.DOM_VK_PAGE_UP || keyCode === KeyEvent.DOM_VK_PAGE_DOWN ||
            keyCode === KeyEvent.DOM_VK_RETURN ||
            keyCode === KeyEvent.DOM_VK_ESCAPE

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

            $ul.on("click", "li.inlinemenu-item", function (e) {
                // Don't let the click propagate upward (otherwise it will
                // hit the close handler in bootstrap-dropdown).
                e.stopPropagation();
                if (self.handleSelect) {

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

        if (this.selectedIndex !== -1) {
            $(items[this.selectedIndex]).find("a").removeClass("highlight");
        }

• Read up
  Read up