Linha do tempo sobrepondo "Tópicos Sugeridos"

Olá.
Em tópicos curtos, a linha do tempo está sobrepondo a seção “Tópicos Sugeridos”:


Acho que a linha do tempo poderia ser ocultada em tópicos sem nenhuma resposta, ou a altura mínima do contêiner de postagens deveria ser aproximadamente a mesma da linha do tempo?

4 curtidas

Se me recordo bem, isso já existe há algum tempo e é comum em telas curtas e tópicos de uma única linha, mas talvez tenhamos regredido? :thinking:

Isso também acontece quando o conteúdo é suficientemente longo e há algum conteúdo acima da postagem

Definitivamente vendo isso.

@awesomerobot me lembrou que costumávamos ocultar a linha do tempo para tópicos de post único, certo? Talvez seja por isso que estamos vendo mais isso, ou talvez seja uma regressão real? De qualquer forma, todos os elementos da linha do tempo estão duplicados nos controles na parte inferior do primeiro (e único) post neste caso…

1 curtida

Sim, acho que temos estado escondendo isso desde 2016: UX: Hide all timeline controls when fewer than 3 posts · discourse/discourse@b9f8264 · GitHub

Até algum tempo atrás, a linha do tempo ocultava

@j.jaffeux, sua mudança recente poderia ter causado esse regresso?

3 curtidas

Definitivamente, devemos ocultar a linha do tempo quando houver apenas UMA postagem. Três parece um pouco demais. Mas uma…

1 curtida

O enorme problema com uma única postagem é que, quando ela é realmente útil como esta, não funciona corretamente. Não suportamos navegação no meio de uma postagem gigantesca, o que torna tudo muito, muito, muito, muito confuso. Muito pior do que apenas o problema de renderização.

2 curtidas

Faça isso com base no tamanho da postagem mais a postagem única, então… não precisamos de uma linha do tempo vertical para uma postagem única de uma frase em um único tópico, precisamos?

Absolutamente não, mas precisamos corrigir a implementação, acho que é isso que estou tentando dizer.

Se eu tivesse que escolher entre removê-la incondicionalmente ou mantê-la para posts únicos muito longos e atípicos, eu escolheria removê-la incondicionalmente.

Mas, se corrigirmos a implementação subjacente, eu apoiaria totalmente mantê-la para o incrível guia do @Johani; seria super útil.

A lógica deve ser

se houver apenas uma postagem no tópico E se essa única postagem tiver menos de .. 1000* caracteres .. suprima completamente a linha do tempo

Até mesmo duas postagens de uma frase cada provavelmente é seguro, talvez?

  • mas e se forem todos caracteres únicos e quebras de linha (CR/LF) etc? Não sei. Acho que uma verificação aproximada é suficiente, não precisamos nos preocupar com casos extremos insanos aqui.
1 curtida

Claro, podemos fazer algo aqui, mesmo usando apenas CSS. Mas teremos que corrigir a barra de rolagem se seguirmos essa abordagem.

Caso contrário, poderíamos simplesmente mudar para se 1 post, então esconder a timeline, o que é ainda mais simples.

Exceto se aquele único post for uma aberração de 30.000 caracteres, não queremos que fique oculto, então… não.

Nós realmente queremos que isso fique oculto… desculpe… esse é o meu ponto. A menos que resolvamos esse problema:

Arrasto a linha do tempo para baixo:

Ela continua no topo. :crying_cat_face:

Essa barra de rolagem, no entanto, funciona como esperado se eu usar a nativa do navegador:

Se deixarmos isso em tópicos com um único post gigante, simplesmente não funciona corretamente, o que é muito confuso. Então, precisamos corrigir isso.

1 curtida

Entendi, precisamos de bons casos de teste informais para garantir que cobrimos aqueles que nos importam.

1 curtida

Não, este não é o commit.

O commit relacionado é muito provavelmente:

Portanto, antes deste commit, a lógica estava meio que duplicada e diferente, e também causava um widget de timeline-controls vazio (mas visível).

Vou dar uma olhada hoje.

3 curtidas

Minha tentativa de corrigir a sobreposição da linha do tempo está aqui:

Isso não resolve o que você explicou: que clicar ou arrastar na linha do tempo não altera a posição de rolagem da janela. Analisei rapidamente como fazer isso e acho que isso deveria ser um projeto dedicado; sempre foi assim e mudar isso tem MUITAS implicações.

Se tivermos apenas uma postagem, agora verificarei a altura da postagem e ocultarei o rolagem se for inferior a 1000px.

Isso representa uma postagem de aproximadamente este tamanho:

Exemplo com uma postagem curta:

Esse commit também corrige outro bug, que fazia o botão de administrador do tópico aparecer em uma posição incorreta ao atualizar a página inteira:

Como este:

Em vez desta posição após a correção:

Também verifiquei se duas postagens com a quantidade mínima de conteúdo não estão sobrepostas:

Nada deve ter mudado no mobile.

4 curtidas

Excelente, obrigada, Joffrey :smiling_cat_with_heart_eyes:

2 curtidas

Isso já foi mesclado e pode ser visto em ação neste tópico, por exemplo (pelo menos enquanto houver apenas uma postagem):

3 curtidas

Este tópico foi automaticamente fechado após 5 dias. Novas respostas não são mais permitidas.