Make photo credit badges consistently clickable
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 68/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- html
- Área
- frontend
Línea de trabajo
Inspecciona los badges de crédito fotográfico en index.html y coffee.html, comparando los spans own-photo con los enlaces Unsplash existentes. Actualiza los badges own-photo para que apunten a photographer.html; después, verifica que todos los badges sean clicables y decide si es necesario ajustar el estilo invert y aria-label.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Photo credit badges across the site come in two shapes:
- Unsplash photos use an
<a>linking to the photo or photographer, e.g. the four onindex.htmland the three oncoffee.html. - William's own photos use a
<span>with the same styling and atitle="Photograph by William Lay", so the badge looks identical but does nothing. Currently the photographer spotlight onindex.htmland the book and French press spotlights oncoffee.html.
A reader cannot tell the difference until they try to click, and a badge that looks like a link but is inert reads as an unfinished placeholder rather than a deliberate credit.
Proposal: make the own-photo badges link to photographer.html, so every badge on the site is clickable and the credit clearly points at a person on purpose.
Worth deciding at the same time:
- Whether the black pill is the right treatment inside
invertspotlights. On a black section the pill disappears and only the white camera icon and name remain. It is legible and already the behaviour onindex.html, but it is accidental rather than designed. - Whether the badge needs an
aria-label, since the camera glyph is decorative and the text alone reads as a bare name.
Split out of #181, where the own-photo badges were first added.
- Lenguaje dominante
- HTML
- Estrellas
- 0
- Forks
- 0
- Merge medio
- 9 h 52 min
- PR fusionados (30 d)
- 59
Preparar el entorno
Aún no hemos revisado los archivos de configuración de este proyecto. Empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de laywill/laywill.github.io
-
design
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
laywill/laywill.github.io#186 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 Medio día Aptitud para principiantes 74/100
laywill/laywill.github.io#135 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
laywill/laywill.github.io#106 ·
Los mantenedores suelen responder en 1 día
-
infra needs-william
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
laywill/laywill.github.io#35 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
infra
Dificultad 3/5 1-2 días Aptitud para principiantes 65/100
laywill/laywill.github.io#187 ·
Los mantenedores suelen responder en 1 día
Todos los issues de laywill/laywill.github.io
Issues similares
-
accepted bug small
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
refined-github/refined-github#10133 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
coder/xum#4739 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
lichess-org/lila#21826 ·
Los mantenedores suelen responder en 1 día
-
dfm.ruAbiertoN: AdGuard for Mac P4: Low T: Social Widget
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
AdguardTeam/AdguardFilters#242670 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día