devstaff-crete/DevStaff-Heraklion

Avoid deeply nested control flow statements.

Open

          if (hist.historyId === currentView.historyId) {
            direction = DIRECTION_FORWARD;

          } else if (currentView.historyId !== hist.historyId) {
            direction = DIRECTION_ENTER;

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

Open

        addEventListener: function( type, listener, useCapture ) {
            if( 'addEventListener' in window ) {
                ( dom.wrapper || document.querySelector( '.reveal' ) ).addEventListener( type, listener, useCapture );
            }
        },

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

        if (initHeight) viewportProperties.height = '0';

Avoid deeply nested control flow statements.

Open

              if (!foundHistoryId) {
                return true;
              }

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

Open

                section.outerHTML = slidify( getMarkdownFromSlide( section ), {
                    separator: section.getAttribute( 'data-separator' ),
                    verticalSeparator: section.getAttribute( 'data-separator-vertical' ),
                    notesSeparator: section.getAttribute( 'data-separator-notes' ),
                    attributes: getForwardedAttributes( section )

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

          } else if (scrollTop > maxScrollTop) {

            scrollTop = maxScrollTop;

          } else {

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

Open

    cancel: function() {
      if (this.host.cancel) {
        this.host.cancel();
      }
      this._resolve(false);

• Read up
  Read up

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

Open

        removeEventListener: function( type, listener, useCapture ) {
            if( 'addEventListener' in window ) {
                ( dom.wrapper || document.querySelector( '.reveal' ) ).removeEventListener( type, listener, useCapture );
            }
        },

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

            if (!self.__enableScrollX && self.__refreshHeight != null) {

              // hide the refresher when it's behind the header bar in case of header transparency
              if (scrollTop < 0) {
                self.__refreshHidden = false;

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

Open

        heightScale = Math.max(self.__indicatorY.minScale, (self.__indicatorY.size - heightDiff) / self.__indicatorY.size);

• Read up
  Read up

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

Open

                            section.outerHTML = slidify( xhr.responseText, {
                                separator: section.getAttribute( 'data-separator' ),
                                verticalSeparator: section.getAttribute( 'data-separator-vertical' ),
                                notesSeparator: section.getAttribute( 'data-separator-notes' ),
                                attributes: getForwardedAttributes( section )

• Read up
  Read up

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

Open

        widthScale = Math.max(self.__indicatorX.minScale,
            (self.__indicatorX.size - widthDiff) / self.__indicatorX.size);

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

          if (tmp && tmp.parentHistoryId === currentView.historyId) {
            direction = DIRECTION_ENTER;
          }

Avoid deeply nested control flow statements.

Open

          if (forwardView && currentView.stateId !== forwardView.stateId &&
             currentView.historyId === forwardView.historyId) {
            // they navigated to a new view but the stack already has a forward view
            // since its a new view remove any forwards that existed
            tmp = getHistoryById(forwardView.historyId);

Function rectContains has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

    rectContains: function(x, y, x1, y1, x2, y2) {

Avoid deeply nested control flow statements.

Open

          for (y = 0; y < splitClasses.length; y++) {
            cssClass = splitClasses[y].trim();
            if (existingClasses.indexOf(' ' + cssClass + ' ') === -1) {
              existingClasses += cssClass + ' ';
            }

Avoid deeply nested control flow statements.

Open

            if (!self.__refreshActive) {
              self.__startDeceleration(timeStamp);
            }

Function resolveState has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

    function resolveState(state, params, paramsAreFiltered, inherited, dst, options) {

Avoid deeply nested control flow statements.

Open

      } else if (version == 7) {
        //iPhone == 7.0 WebView
        viewportProperties.height = DEVICE_HEIGHT;
      }

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

Open

            toArray( slide.querySelectorAll( 'iframe' ) ).forEach( function( el ) {
                el.contentWindow.postMessage( 'slide:stop', '*' );
            });

• Read up
  Read up