TeamNewPipe/NewPipe

Standardize view animation duration

Open

#3.038 aberto em 30 de jan. de 2020

Ver no GitHub
 (11 comments) (3 reactions) (0 assignees)Java (3.577 forks)batch import
GUIfeature requestgood first issue

Métricas do repositório

Stars
 (38.153 stars)
Métricas de merge de PR
 (Mesclagem média 9d 17h) (17 fundiu PRs em 30d)

Description

If you check the usages of animateView you will see that the typical range will be 0 all the way to 500, with most in the 100-250 range.

It should've been done with a default value in the first place. It is pretty much arbitrary right now, it was whatever the person felt like at the time of writing.

I'll leave this task to another PR, which the goal will be to standardize the used duration of animations.

_Originally posted by @mauriciocolli in https://github.com/TeamNewPipe/NewPipe/pull/2309#discussion_r371625549

Guia do colaborador

Direção de pesquisa
Identifique todos os usos de animateView no código, categorize os valores de duração e proponha uma duração padrão ou um conjunto de constantes. Em seguida, implemente a mudança em todas as ocorrências.
Pilha de tecnologia
java
Domain
mobile
Tipo Issue
Refatoração
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.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Android developmentGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70