cse112-sp20/Team-Potato

Function render has 142 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  render() {
    /** Add those to the current state
     * addGroupModel: decide whether the add group modal pop oopen or not
     * activeTabs: the tabs being active currently on chrome browser
     * tabGroups: the tabGroups the user has created

File Menu.jsx has 377 lines of code (exceeds 250 allowed). Consider refactoring.

Open

/**
 * @fileOverview This file is the menu class which contains tabgroups,
 *               activetabs, and savedtabs. Here included functionalities from
 *               constrcutor, component render, create a new tabgroup, and so on
 *

Avoid deeply nested control flow statements.

Open

            if (i !== deleteIndex) {
              updatedTabs.push({
                title: tabGroups[deleteGroup].tabs[i].title,
                url: tabGroups[deleteGroup].tabs[i].url,
                stored: tabGroups[deleteGroup].tabs[i].stored,

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

Open

    } else if (e.target.attributes.getNamedItem('droppable').value !== 'true') {
      /** if not droppable, do nothing */
      e.preventDefault();
      e.dataTransfer.effectAllowed = 'none';
      e.dataTransfer.dropEffect = 'none';

• Read up
  Read up

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

Open

    if (
      e.target === undefined ||
      e.target.attributes.getNamedItem('droppable') === null
    ) {
      /** if invalid, do nothing */

• Read up
  Read up