Notifications - "unread only" filter

Certo, vou focar nisso primeiro e enviarei os mockups dessa página em breve :slightly_smiling_face:

5 curtidas


O que acham? @Johani, @jomaxro

4 curtidas

Ooo, gostei! Muito bem feito.

4 curtidas

Então, está finalizado? Posso começar a trabalhar nisso?

Claro, mas certifique-se de usar o select kit ali e não um controle personalizado.

3 curtidas

Claro, vou usar o select kit :slightly_smiling_face:

1 curtida

Usei o select kit e criei um PR.

11 curtidas

Concluí esta tarefa e meu pull request foi mesclado.

7 curtidas

@Ahmed_Gagan

Uma coisa que notei: atualmente, ele não busca todas as notificações não lidas. Ele filtra apenas a primeira “página” carregada delas, mas se houver mais não lidas no passado, você precisa rolar para baixo na visualização “Todos” e só então alternar para as não lidas.

Seria legal se ao alternar para o filtro “Não lidas” ele carregasse todas as notificações não lidas e não apenas as da primeira página, se faz sentido :slight_smile:

EDIT.
Acabei de encontrar outro bug, acho eu.
Quando você clica com o botão do meio nesta visualização:

Ele deveria abrir um tópico em uma nova aba, mas em vez disso limpa a notificação sem abrir nenhuma nova aba.

4 curtidas

Vou testar mais uma vez e retorno para você :slightly_smiling_face:

3 curtidas

Muito animado por ver isso ser implementado!

Obrigado a todos que incubaram isso desde janeiro de 2016 :heart_eyes:

4 curtidas

Comecei a trabalhar nessas alterações, que serão enviadas em breve.

8 curtidas

Acabei de mesclar o PR do @Ahmed_Gagan para isso, e está funcionando muito bem para mim no Meta :heart_eyes:

Vou implantar essa alteração no seu site nas próximas horas, @loginerror.

9 curtidas