-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
style: Titulo para archivos adjuntos de charlas y formato lista para … #542
base: master
Are you sure you want to change the base?
Conversation
…charlas de miembro equipo
Hola @Scot3004
|
1 listas para todos 😜 website/templates/nosotros-miembro.html Line 45 in dc98f89
|
@@ -15,6 +15,7 @@ <h3 class="title">{{ this.title }}</h3> | |||
{{this.details}} | |||
</div> | |||
<div class="attachments"> | |||
{{this.attach}} | |||
<h4>Adjuntos</h4> |
There was a problem hiding this comment.
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> |
There was a problem hiding this comment.
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
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 */ En vez del "a" usa una clase |
Pull request
Ninguno
Observaciones
Mi intención de mejorar la presentación de la información