lbryio/lbry-desktop

spacebar doesnt work flawless with handlers

Open

#3.870 aberto em 21 de mar. de 2020

Ver no GitHub
 (6 comments) (0 reactions) (0 assignees)JavaScript (412 forks)batch import
area: viewerhelp wantedtype: bug

Métricas do repositório

Stars
 (3.596 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

Bug when i press on the spacebar on my keyboard with the miniplayer playing a video, 2 things happen: the page scrolls down, and the video pauses playing in miniview.

To Reproduce as explained above

Expected behavior expected behaviour to me would naturally be in this case: spacebar press to only pause/resume and not scroll the page. maybe use the page up/down keys for that specific?

System Configuration

  • LBRY Daemon version: 0.63.0
  • LBRY App version: 0.43.4
  • LBRY Installation ID: not needed for this(?)
  • Operating system: 64bit based Ubuntu 19.10 Linux. (KDE desktop environment)

Additional context love LBRY alot! just wish i could help fixing the issues i come across :)

Guia do colaborador

Direção de pesquisa
Investigue o ouvinte de eventos keydown para a tecla de espaço no componente miniplayer. Impedir o comportamento de rolagem padrão quando a barra de espaço é pressionada durante a reprodução do vídeo.
Pilha de tecnologia
javascript
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.
1-3 horas
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
JavaScriptDOM event handling
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75