Skip to content

Conversation

@pyjavo
Copy link
Member

@pyjavo pyjavo commented Aug 20, 2024

  • Titulo para archivos adjuntos de charlas
  • formato lista para charlas de miembro equipo

Pull request

Ninguno

Observaciones

Mi intención de mejorar la presentación de la información

@pyjavo pyjavo requested a review from Scot3004 August 20, 2024 18:57
@pyjavo
Copy link
Member Author

pyjavo commented Aug 20, 2024

Hola @Scot3004
Está en draft porque aún tengo dudas de las cosas que tenía en mente para este PR:

  1. La primera es listar las charlas usando etiquetas li de HTML
    Screenshot from 2024-08-20 16-23-49

  2. Mi segunda idea era hacer lo mismo con los bloques de "blog posts" y "proyectos que organiza" pero al estar en 2 columnas, no se me ocurrió una forma para que quedaran chéveres.

    • ¿Qué recomiendas para este punto?
  3. Mostrar la palabra "Adjunto" (y en un futuro "attachments" para inglés) para organizar un poco la información en la vista detalle de una charla (También intenté listarlos en HTML, pero eso también depende de como se editen en markdown)

@Scot3004
Copy link
Member

1 listas para todos 😜
2 también listas, se puede implementar a 2 o a 1 columnas como consideres apropiado
3 de acuerdo, le da mucho más orden tener la palabra
Y en cuanto a cómo colocar la internacionalización hay unos json que tienen las traducciones en varias partes de la página hay ejemplos de estos, para el ejemplo particular hay un json llamado nosotros.json que aplica para el caso puntual
Puedes ver en nosotros miembros en la línea 45 como se usa

<h2>{{bag('nosotros', alt, 'contribuciones')}}</h2>

</div>
<div class="attachments">
{{this.attach}}
<h4>Adjuntos</h4>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En el escenario que no existan anexos el va a mostrar este titulo y luego una sección en blanco, creo que seria bueno verificar si tiene anexos y en caso que no, evitar mostrar esta sección

<div>
<a href="{{event|url}}">{{event.title}}</a>
</div>
<li><a href="{{event|url}}">{{event.title}}</a></li>
Copy link
Member

@Scot3004 Scot3004 Sep 16, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

creo que se vería mejor el código si ajustas el espaciado de este bloque de código

@pyjavo
Copy link
Member Author

pyjavo commented Nov 21, 2024

Revisar https://developer.mozilla.org/es/docs/Web/CSS/::before para los bullet points de proyectos o blog posts

Sergio Orozco says:/* Añade un corazón antes de los enlaces */
a::before {
content: "♥";
padding-right: 40px;}

En vez del "a" usa una clase

@andoresuperesu
Copy link
Collaborator

Tbn svgs: https://stackoverflow.com/questions/19255296/is-there-a-way-to-use-svg-as-content-in-a-pseudo-element-before-or-after

@pyjavo pyjavo marked this pull request as ready for review July 1, 2025 01:10
@pyjavo pyjavo requested a review from Scot3004 July 1, 2025 01:10
@pyjavo
Copy link
Member Author

pyjavo commented Jul 1, 2025

Gracias muchachos, ya implementé los cambios sugeridos. @Scot3004 ya puedes volver a revisar

@Scot3004
Copy link
Member

Scot3004 commented Jul 1, 2025

Estoy observando en el preview deploy que el orden cambio y ahora se muestran las charlas más antiguas primero, es esto esperado?

Así se ve en pybaq.co
Screenshot_2025-07-01-09-55-43-585_com.brave.browser.jpg

Y así se ve ahora

Screenshot_2025-07-01-09-57-00-380_com.android.chrome.jpg

@pyjavo
Copy link
Member Author

pyjavo commented Jul 1, 2025

@Scot3004 si es esperado, es parte de mi propuesta.

Es decir, según mi lógica, tu primera charla en la comunidad está listada en el elemento 1, la siguiente en el elemento 2 y así sucesivamente hasta llegar a tu última charla en la comunidad (la charla 8). Quise darle un orden a cada charla y pues para que fuera fácil de contar 😅 ¿Tiene sentido?

@pyjavo pyjavo merged commit 9e9b008 into PyBAQ:master Jul 1, 2025
5 checks passed
@pyjavo pyjavo deleted the estilos-propuestos branch July 1, 2025 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants