electerious / electerious/basicLightbox

<picture> tag support

Ouverte
#37 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
JavaScript
Étoiles
604
Forks
57
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

I am trying to add support for webp image formats with jpg fallbacks. When I write my listener like this the lightbox works as intended:

```
window.addEventListener('load', (event) => {
document.querySelectorAll('.modal').forEach(image => {
image.addEventListener('click', e => {
basicLightbox.create(`

`).show();
});
});
});
```

However, when I write my listener like this, the lightbox does not close when I click on the image, only when I click on the background, making the lightbox difficult to close:

```
window.addEventListener('load', (event) => {
document.querySelectorAll('.modal').forEach(image => {
image.addEventListener('click', e => {
basicLightbox.create(`





`).show();
});
});
});
```

I imagine this has something to do with the `` tag. Any thoughts on why this would be happening? Thanks

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Piste de recherche

Reproduisez les deux variantes de listeners de l’issue, en comparant le markup img simple avec le markup picture/source/img. Commencez par basicLightbox.create(...).show() et suivez la manière dont les clics sur le contenu affiché et l’arrière-plan sont gérés. Le travail est terminé lorsque la lightbox de fallback WebP basée sur picture se ferme quand on clique sur l’image, comme le fait la version avec img simple.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
javascript
Domaine
frontend
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
À clarifier
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.