Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

Zooming Images which don't match the hardcoded thumbnail aspect ratio cause problems

Aperta
  • Qualcosa non va?
#31 1 commento 0 reazioni 1 assegnatario Vedi su GitHub

@zachwise ci sta già lavorando.

Dal 4/9/2015.

Valutazione

Questa issue non è ancora stata valutata.

Descrizione

The autogenerated thumbnails of the zooming images are 313x207 (based on the original sketches)

Some example Zooming Images don't match that aspect ratio, which means that the thumbnail is visible when the image is zoomed:
image

We could probably handle this in part by hiding the thumbnail when the large image is produced, but also, is it problematic to have thumbnails which cut off part of the original image? I'm not sure if the current mechanism for automatically generating the thumbnails can be made to preserve the aspect ratio.

Is there a design rationale for enforcing a common size for all zooming image thumbnails? I'm guessing that people will want flexibility, but if any of this is going to be automated, we need to pin it down. (Alternatively, people can use the media browser to upload both forms of the image and put the URLs in directly...)

Lingua principale
HTML
Stelle
9
Fork
6
Nessuna PR unita negli ultimi 30g

Preparare l'ambiente

Non abbiamo ancora controllato i file di configurazione di questo progetto. Parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di NUKnightLab/learncms

Tutte le issue di NUKnightLab/learncms

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.