SSENSE/vue-carousel

Bug: Adjustable height is broken on both Firefox and IE11

Offen

#412 geöffnet am 29.03.2019

Auf GitHub ansehenContributor Guide
 (8 Kommentare) (1 Reaktion) (0 zugewiesene Personen)JavaScript (513 Forks)batch import
buggood first issuehelp wanted

Repository-Metriken

Stars
 (1.705 Sterne)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

Bug Report

Current Behavior When using adjustable height, Firefox seems to present the carousel as having a height of only 48. If you click on the carousel after it loads, it will instantly resize to the proper height.

Input Code and steps to reproduce

  • set :adjustableHeight="true"
  • refresh

Expected behavior/code The carousel height should initialize at the height of the first item. It works in Chrome but in FF/IE11 it initializes at 24 or 48 height.

Environment

  • Browsers: Firefox / IE11
  • OS: OSX 10.14.3

Possible Solution After debugging it looks like it has something to do with getCarouselHeight().

Contributor Guide

Research-Richtung
Überprüfen Sie die Funktion getCarouselHeight() in der Komponente und untersuchen Sie, wie sie die Höhe des ersten Elements berechnet. Testen Sie die Funktion in Firefox und IE11, um browserspezifische Eigenheiten zu identifizieren (z. B. Unterschiede bei offsetHeight, clientHeight). Erwägen Sie die Verwendung eines ref oder Lifecycle Hooks, um sicherzustellen, dass der Karussell vollständig gerendert ist, bevor die Höhe berechnet wird.
Tech Stack
javascriptvue
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitVue.js
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60