SSENSE/vue-carousel

Bug: Adjustable height is broken on both Firefox and IE11

Aberta

#412 aberto em 29 de mar. de 2019

Ver no GitHubGuia do colaborador
 (8 comentários) (1 reação) (0 responsável)JavaScript (513 forks)batch import
buggood first issuehelp wanted

Métricas do repositório

Stars
 (1.705 estrelas)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

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().

Guia do colaborador

Direção de pesquisa
Inspecione a função getCarouselHeight() no componente e investigue como ela calcula a altura do primeiro item. Teste a função no Firefox e no IE11 para identificar peculiaridades específicas do navegador (por exemplo, diferenças em offsetHeight, clientHeight). Considere usar um ref ou um hook de ciclo de vida para garantir que o carrossel esteja totalmente renderizado antes de calcular a altura.
Pilha de tecnologia
javascriptvue
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitVue.js
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60