Acceso rápido a marcadores y mensajes en el menú de usuario

Ya está hecho :smiley:

Me parece una buena idea; añadir un outlet de plugin allí debería ser bastante sencillo.

5 Me gusta

Ah, no había instalado el componente del tema aún, así que no podía ver la pestaña asignada. Ahora lo tengo y veo que está ahí. ¡Genial! :candy:

4 Me gusta

Eché un vistazo rápido y no creo que esté relacionado con el componente. También puedo reproducir el icono gris en try:

Pero el icono es azul en modo seguro, tanto para meta como para try… Qué extraño.

7 Me gusta

Sí, estoy de acuerdo con esto. Podemos eliminar el engranaje. Hay otras formas de acceder a él y no es crítico que esté aquí.

Creo que deberíamos esperar con ese aspecto; esto ya es un cambio bastante radical. Deberíamos vivir con él por un tiempo antes de agregar aún más cambios.

En general, me gusta mucho :+1:

7 Me gusta

Yo también noté esto en community.wanikani.com, así que no parece estar relacionado conmigo.


Este cambio es realmente genial hasta ahora. Me gusta que haga que los marcadores sean más visibles, ya que antes era muy fácil olvidarse de ellos.

6 Me gusta

He eliminado el engranaje y aumentado un poco el ancho:

No estoy seguro de si queremos aún más (¿el de arriba?)

En realidad, ya he realizado el cambio, pero estoy dispuesto a revertirlo o dejarlo como un componente sin integrarlo en el núcleo más adelante.

Por si acaso, este es otro diseño que tenía en mente, pero podemos dejar la conversación para más adelante cuando estemos considerando esto.

(La nueva pestaña de perfil depende de esta PR).

10 Me gusta

En realidad, tengo un comentario. Al menos en el tema oscuro para móviles, resulta bastante difícil ver qué pestaña está seleccionada. Si miro de cerca, puedo ver la línea que rodea el icono de la pestaña seleccionada, pero es muy sutil. Quizás el color de la línea debería tener un poco más de contraste con el fondo. De hecho, el color de la línea que veo rodeando el cuadro de respuesta mientras escribo esta respuesta (sigue siendo el tema oscuro para móviles) podría funcionar muy bien.

5 Me gusta

¡Esto es FANTÁSTICO :confetti_ball: … Mi siguiente comentario va a ser muy minucioso… así que por favor… ten paciencia conmigo. :bear:

  1. Por alguna razón… ahora tengo una barra de desplazamiento, así que estamos unos píxeles fuera. Sé que @Johani y @awesomerobot han estado luchando un poco con el dimensionamiento dinámico aquí, así que esto puede no ser totalmente nuevo.

  1. Tenemos que mover lo de modo anónimo a tu nuevo y maravilloso hogar (configuración del sitio “permitir publicaciones anónimas”):

  1. Las pestañas parecen tener orejas… OK, esto es lo más minucioso de todos, pero 4px ahí ayudarán a que parezca más una pestaña.

  1. La flecha hacia abajo no tiene sentido aquí, solo añade ruido:

Mi sensación es que esto está listo para lanzarse :ship:, vivamos con esto durante, digamos, 1 semana más (pendiente de la central de críticas minuciosas que acabo de ofrecer). Luego, este componente de vida corta podría tener que morir en favor de simplemente lanzarlo con el núcleo.

Cuanto más lo uso, más me encanta. Hace que los marcadores y la asignación sean mucho más útiles para mí.

17 Me gusta

¡Me encanta este cambio!

Tuve una sugerencia: ¿no deberían los mensajes aparecer después de las notificaciones, ya que se usan con más frecuencia que los marcadores en general?

2 mejoras menores que creo que serían buenas:

1) Cuando entras a la página de perfil, las pestañas superiores son:

Creo que sería bueno añadirlas (de hecho, solo Notificaciones, Invitaciones e Insignias aún no están) a la pestaña de perfil (es decir, sam), en este orden (y eliminando la flecha hacia abajo, que no tiene sentido aquí).

2) Cuando pasas el cursor sobre la flecha hacia abajo en todas las pestañas, muestra “ver notificaciones anteriores”, pero tendría más sentido que mostrara “ver marcadores anteriores” y “ver mensajes anteriores” en sus respectivas pestañas.

Este componente del tema es genial hasta ahora. ¡Gracias por hacerlo!

1 me gusta

Creo que esto depende de la longitud de los títulos de los temas en el menú desplegable, lo cual es muy variable. Una forma de “solucionarlo” es restringir todos los títulos a lo que quepa en una sola línea y ocultar el desbordamiento, pero sería bastante doloroso de ver. Sin embargo, ¿restringirlos a solo dos líneas parece más viable según lo que tengo en mi menú desplegable ahora mismo?

7 Me gusta

+1. Yo también tengo una barra de desplazamiento, con o sin el nuevo componente.

Esto se debe a cómo estimamos actualmente la cantidad de notificaciones que se muestran. Si la altura total supera la altura de la ventana (en el caso de múltiples notificaciones de tres líneas), la barra de desplazamiento será visible.

Genial, probemos esto en el componente. Añadí algunos estilos para lograr exactamente eso.

7 Me gusta

:white_check_mark:

:ear: Opté por el enfoque inferior (¿más fácil y bonito?) para empujar los elementos un poco hacia adentro. Pero me di cuenta de que el icono de la persona ya no se alinea bien con los iconos de abajo:

¿Estamos de acuerdo con esto? Mientras tanto, intentaré encontrar una solución limpia para esto.

¡Lo eliminé para la pestaña de perfil!

Además, la flecha siempre se muestra para asignaciones, marcadores y mensajes, independientemente del número de elementos. Me preocupaba que la función de “segundo clic para ir a la página completa” no fuera lo suficientemente obvia.

¡Ah, gracias, buen hallazgo! ¿O quizás un genérico “ver todo” serviría? Dejaré esto (y otro vistazo al uso de etiquetas) para cuando lo integremos en el núcleo.

Tiene sentido, ¡gracias! Ajusté un poco el color para el modo oscuro.

13 Me gusta

¡Esto se ve fantástico para mí! Digo que lo mantengamos así hasta el próximo miércoles en meta. Entonces… ¡una PR es absolutamente bienvenida para integrar esto tal cual en el núcleo y eliminar el componente! @codinghorror también lo ama.

No me importa la ligera desalineación debida a la oreja; se ve bien para mí, y normalmente este tipo de cosas me molestan mucho.

¡Me encanta la velocidad de iteración aquí, la calidad y el esfuerzo que has puesto en ello!

En este punto, la única “pequeña observación” que me queda es que, en la carga inicial absoluta, los elementos saltan un poco y aparece una barra de desplazamiento durante unas fracciones de segundo. Es muy menor y estoy seguro de que un poco de CSS puede solucionarlo.

12 Me gusta

Un error: al hacer clic en Cerrar sesión, te lleva a tu resumen de usuario y no puedes cerrar sesión correctamente:

6 Me gusta

Puedo confirmar el informe de error de @nexo aquí en meta.

En mis sitios, aún no parece tener este menú person tobiaseigen; al hacer clic en él, simplemente me lleva a mi página de actividad.

3 Me gusta

Gracias, ¡acabo de aplicar una solución! Actualizar el componente del tema debería solucionarlo.

¿Podrías intentar actualizar Discourse? Necesitarás tener la versión más reciente de Discourse para la pestaña de acceso rápido de persona/perfil, específicamente este commit de hace dos días.

8 Me gusta

¡Qué rápido! ¡La solución funciona!

¡Argh! Debería haber sabido que eso era lo primero que debía probar. :slight_smile: Pero sí, actualizar Discourse funcionó y ahora esta funcionalidad funciona en mi sitio.

¡Gracias! :rocket:

6 Me gusta

¿Qué tal si se incluye una configuración de enlaces personalizados en el componente del tema? Creo que esto sería súper útil para cualquier sitio con SSO, para proporcionar enlaces al panel de control relevantes solo para los miembros conectados. En mi sitio, simplemente reemplacé un encabezado personalizado con el Brand Header, que funciona de maravilla y, además, es futuro sin necesidad de personalizaciones. Ahora también me gustaría poder añadir un “Editar perfil” que enlace a WordPress, que también sea a prueba de futuro.

3 Me gusta

Puedes hacer que todas las notificaciones bajo las pestañas sean desplazables. Se está redirigiendo a otra página incluso para una rápida ojeada.