Notifications - "unread only" filter

Vale, me centraré en eso primero y te enviaré pronto los bocetos de esa página :slightly_smiling_face:

5 Me gusta


¿Qué opinan? @Johani, @jomaxro

4 Me gusta

¡Ooo, me encanta! Muy bien hecho.

4 Me gusta

¿Entonces es definitivo? ¿Puedo empezar a trabajar en ello?

Claro, pero asegúrate de usar el kit de selección allí y no un control personalizado.

3 Me gusta

Claro, usaré el kit de selección :slightly_smiling_face:

1 me gusta

He utilizado select kit y he creado un PR.

11 Me gusta

He completado esta tarea y mi pull request ha sido fusionado.

7 Me gusta

@Ahmed_Gagan

Una cosa que noté: actualmente no carga todas las notificaciones no leídas. Solo filtra la primera ‘página’ cargada, pero si tienes más no leídas antiguas, tienes que desplazarte hacia abajo en la vista de Todo y solo entonces cambiar a las no leídas.

Sería genial si al cambiar al filtro No leídas se cargaran todas las notificaciones no leídas y no solo las de la primera página, si me entiendes :slight_smile:

EDIT.
Acabo de descubrir otro error, creo.
Cuando haces clic con el botón del medio en esta vista:

Debería abrir un tema en una pestaña nueva, pero en su lugar borra la notificación sin abrir ninguna pestaña nueva.

4 Me gusta

Lo probaré una vez más y te responderé :slightly_smiling_face:

3 Me gusta

¡Muy emocionado de ver esto implementado!

Gracias a todos los que han estado incubando esto desde enero de 2016 :heart_eyes:

4 Me gusta

Se han comenzado a trabajar estos cambios y serán enviados pronto.

8 Me gusta

Acabo de fusionar la PR de @Ahmed_Gagan para esto, y me está funcionando muy bien en Meta :heart_eyes:

Ese cambio se desplegará en tu sitio en las próximas horas, @loginerror

9 Me gusta