Another little code question (variable: top)

Ouverte
#18 1 commentaire 0 réactions 1 personne assignée Voir sur GitHub

@tylersticka y travaille déjà.

Depuis le 24/4/2014.

Évaluation

Cette issue n'a pas encore été évaluée.

Description

question

Hiya!

This line in lib/simpleslideview.js: https://github.com/cloudfour/SimpleSlideView/blob/master/lib/simpleslideview.js#L266

Is in the beforeChangeEnd callback:

      beforeChangeEnd = function() {
        if (_this.options.resizeHeight) {
          if (_this.options.maintainViewportHeight && window.innerHeight > _this.lastViewportHeight) {
            _this.lastViewportHeight = window.innerHeight;
            $html.css('min-height', (_this.lastViewportHeight + top) + 'px');
          }
          return _this.$container.animate({
            height: outerHeight($targetView)
          }, _this.options.heightDuration, _this.options.easing, onChangeEnd);
        } else {
          return onChangeEnd();
        }
      };

top isn't defined. Not sure what it should be so need a bit of input to fix it!

Langage dominant
HTML
Étoiles
257
Forks
31
Métriques de merge des PR
Aucune PR mergée en 30 j

Préparer son environnement

Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de cloudfour/SimpleSlideView

Toutes les issues de cloudfour/SimpleSlideView

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.