supporto per image-post e image-list #340
3 Participants
Notifications
Due Date
No due date set.
Blocks
Depends on
#330 Larghezza e centratura immagini
BgLUG/bglug.it
#342 immagine copertina gallery
BgLUG/bglug.it
Reference: BgLUG/bglug.it#340
Loading…
x
Reference in New Issue
Block a user
No description provided.
Delete Branch "paspo/bglug.it:big-images"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
fixes #276
Le gallerie non hanno più l'anteprima.
Bisogna mettere
image: IMG_20241026_113926.jpg
nel front mattersupporto per image-post e image-listto wip: supporto per image-post e image-listho rimosso la cosa di impostare la prima immagine disponibile se non ne è stata definita una.
si può risolvere in entrambi i modi: esplicitando l'immagine da usare come anteprima nel front-matter o riabilitando la cosa del prenderne una a caso.
personalmente preferirei scegliere quale immagine usare
d'accordo.
ma non si dovrebbe usare image-post o image-list?
image non è deprecato?
infatti
deprecato ma supportato
wip: supporto per image-post e image-listto supporto per image-post e image-listfc9066bfd5
to1aacef7869
Perchè non si unisce automaticamente dopo l'approvazione?
perchè non l'ho impostato.
vedi #342 (comment)
eppure compare questo messaggio
paspo ha programmato questa pull request per unire automaticamente quando tutti i controlli hanno successo...
cmq meglio, considerato che prima si approva #342
@ -37,0 +11,4 @@
{{- if .Params.image_list }}
{{- $image_src = .Params.image_list.src }}
{{- with .Params.image_list.alt }}
{{- $image_alt = . }}
Perché hai usato with invece che fare
$image_alt = .Params.image_list.src
(come sopra)?Imo se il significato è equivalente (non conosco bene
hugo
), allora sarebbe meglio usare quello che ho suggerito per maggiore chiarezza e meno linee da mantenerewith
skippa il codice nestato se l'elemento non è disponibile.In questo modo,
alt
ecaption
non sono obbligatori.@ -37,0 +14,4 @@
{{- $image_alt = . }}
{{- end }}
{{- with .Params.image_list.caption }}
{{- $image_caption = . }}
idem sopra
@ -39,0 +38,4 @@
{{- with .Params.image_caption }}
{{- $image_caption = . }}
{{- end }}
{{- end }}
Come viene gestito il caso di fallback?
Ah, ok. Forse è meglio spostare
{{- $image := (resources.Get "/images/placeholder_elenco.png") -}}
qua sotto, in un else, per ragioni di ordine mentale.qui però sfociamo nella semantica....
Il default si imposta all'inizio e se ne fa override in seguito nel codice.
Il fallback si imposta alla fine se non è stato impostato un altro valore nel codice.
A mio avviso, è la stessa roba, dipende solo da come la vuoi chiamare.
Onestamente, siamo io e te a guardare a sto codice, e anche se non lo hai trovato subito, è stata questione di secondi, immagino...
Personalemnte, non starei a perderci il sonno.
Se proprio ti pesa, vai di PR.
@ -28,0 +31,4 @@
{{- if .Params.image_post }}
{{- $image_src = .Params.image_post.src }}
{{- with .Params.image_post.alt }}
{{- $image_alt = . }}
idem sopra
@ -28,0 +34,4 @@
{{- $image_alt = . }}
{{- end }}
{{- with .Params.image_post.caption }}
{{- $image_caption = . }}
idem
@ -28,0 +42,4 @@
{{- if and (not $image_src) .Params.image }}
{{- $image_src = .Params.image }}
{{- with .Params.image_alt }}
{{- $image_alt = . }}
idem
@ -28,0 +45,4 @@
{{- $image_alt = . }}
{{- end }}
{{- with .Params.image_caption }}
{{- $image_caption = . }}
idem
@ -51,3 +60,2 @@
{{- end -}}
{{- with .Params.image_caption -}}
<figcaption style="max-width: {{- $img.Width -}}px;">{{- . | markdownify -}}</figcaption>
<img src="{{ $image.RelPermalink }}" alt="{{ $image_alt }}" {{- if ge (mul $image.Height $ratio) $image.Width -}}style="width: auto;" {{- end -}}/>
Non mi piace che viene aperto un blocco codice hugo con if e chiuso con end nella stessa linea.
Non c'è qualche trucco che permette di ridurre la lunghezza del codice o renderlo più leggibile?, tipo usando l'equivalente echo di php (in php avrei fatto qualcosa tipo ` $image.Width) echo "style=\"width: auto;\"" ?>)
E comunque compattando l'espressione della condizione.
hai ragione sul fatto che sia meno leggibile, però è comodo e viene usato spesso nei temi e se non ricordo male c'è anche negli esempi di hugo
@ -7,2 +7,3 @@
{{- $image_caption := "" }}
{{/* immagine di default */}}
{{- $featimg := (resources.Get "/images/placeholder_autore.png").RelPermalink -}}
{{- $image := (resources.Get "/images/placeholder_autore.png") -}}
idem
@ -38,0 +12,4 @@
{{- if .Params.image_list }}
{{- $image_src = .Params.image_list.src }}
{{- with .Params.image_list.alt }}
{{- $image_alt = . }}
idem
@ -38,0 +15,4 @@
{{- $image_alt = . }}
{{- end }}
{{- with .Params.image_list.caption }}
{{- $image_caption = . }}
idem
@ -40,0 +23,4 @@
{{- if and (not $image_src) .Params.image_post }}
{{- $image_src = .Params.image_post.src }}
{{- with .Params.image_post.alt }}
{{- $image_alt = . }}
idem
@ -40,0 +34,4 @@
{{- if and (not $image_src) .Params.image }}
{{- $image_src = .Params.image }}
{{- with .Params.image_alt }}
{{- $image_alt = . }}
idem
@ -40,0 +37,4 @@
{{- $image_alt = . }}
{{- end }}
{{- with .Params.image_caption }}
{{- $image_caption = . }}
idem
@ -40,0 +43,4 @@
{{- if $image_src }}
{{- $image = .Page.Resources.GetMatch $image_src }}
{{- end}}
idem (fallback qui sotto)
1aacef7869
to3e313a4cb8
secondo me ha rimosso la cosa quando gli ho esplicitato #342 come dipendenza
Giusto! Interessante funzionalità.