supporto per image-post e image-list #340

Merged
paspo merged 1 commits from paspo/bglug.it:big-images into main 2024-11-12 13:08:18 +00:00
Owner

fixes #276

fixes #276
paspo added 1 commit 2024-11-10 18:05:16 +00:00
supporto per image-post e image-list
All checks were successful
Linting markdown on PR / unicode-check (pull_request) Successful in 2m50s
fc9066bfd5
paspo scheduled this pull request to auto merge when all checks succeed 2024-11-10 18:05:25 +00:00
paspo added a new dependency 2024-11-10 18:08:33 +00:00
Member

Le gallerie non hanno più l'anteprima.

Le gallerie non hanno più l'anteprima.
Member

Bisogna mettere image: IMG_20241026_113926.jpg nel front matter

Bisogna mettere `image: IMG_20241026_113926.jpg` nel front matter
paspo changed title from supporto per image-post e image-list to wip: supporto per image-post e image-list 2024-11-11 10:45:53 +00:00
Author
Owner

Bisogna mettere image: IMG_20241026_113926.jpg nel front matter

ho 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

> Bisogna mettere `image: IMG_20241026_113926.jpg` nel front matter ho 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
Member

d'accordo.
ma non si dovrebbe usare image-post o image-list?
image non è deprecato?

d'accordo. ma non si dovrebbe usare image-post o image-list? image non è deprecato?
Author
Owner

d'accordo.
ma non si dovrebbe usare image-post o image-list?

infatti

image non è deprecato?

deprecato ma supportato

> d'accordo. > ma non si dovrebbe usare image-post o image-list? infatti > image non è deprecato? deprecato ma supportato
paspo added a new dependency 2024-11-11 20:58:08 +00:00
paspo changed title from wip: supporto per image-post e image-list to supporto per image-post e image-list 2024-11-11 21:12:45 +00:00
paspo force-pushed big-images from fc9066bfd5 to 1aacef7869 2024-11-11 21:16:14 +00:00 Compare
omarl approved these changes 2024-11-12 07:28:47 +00:00
Member

Perchè non si unisce automaticamente dopo l'approvazione?

Perchè non si unisce automaticamente dopo l'approvazione?
Author
Owner

Perchè non si unisce automaticamente dopo l'approvazione?

perchè non l'ho impostato.

vedi #342 (comment)

> Perchè non si unisce automaticamente dopo l'approvazione? perchè non l'ho impostato. vedi https://git.libreschool.org/BgLUG/bglug.it/pulls/342#issuecomment-3945
Member

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

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
amreo requested changes 2024-11-12 12:10:02 +00:00
@ -37,0 +11,4 @@
{{- if .Params.image_list }}
{{- $image_src = .Params.image_list.src }}
{{- with .Params.image_list.alt }}
{{- $image_alt = . }}
Collaborator

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 mantenere

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 mantenere
Author
Owner

with skippa il codice nestato se l'elemento non è disponibile.
In questo modo, alt e caption non sono obbligatori.

`with` skippa il codice nestato se l'elemento non è disponibile. In questo modo, `alt` e `caption` non sono obbligatori.
paspo marked this conversation as resolved
@ -37,0 +14,4 @@
{{- $image_alt = . }}
{{- end }}
{{- with .Params.image_list.caption }}
{{- $image_caption = . }}
Collaborator

idem sopra

idem sopra
paspo marked this conversation as resolved
@ -39,0 +38,4 @@
{{- with .Params.image_caption }}
{{- $image_caption = . }}
{{- end }}
{{- end }}
Collaborator

Come viene gestito il caso di fallback?

Come viene gestito il caso di fallback?
Collaborator

Ah, ok. Forse è meglio spostare {{- $image := (resources.Get "/images/placeholder_elenco.png") -}} qua sotto, in un else, per ragioni di ordine mentale.

Ah, ok. Forse è meglio spostare `{{- $image := (resources.Get "/images/placeholder_elenco.png") -}}` qua sotto, in un else, per ragioni di ordine mentale.
Author
Owner

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.

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.
paspo marked this conversation as resolved
@ -28,0 +31,4 @@
{{- if .Params.image_post }}
{{- $image_src = .Params.image_post.src }}
{{- with .Params.image_post.alt }}
{{- $image_alt = . }}
Collaborator

idem sopra

idem sopra
paspo marked this conversation as resolved
@ -28,0 +34,4 @@
{{- $image_alt = . }}
{{- end }}
{{- with .Params.image_post.caption }}
{{- $image_caption = . }}
Collaborator

idem

idem
paspo marked this conversation as resolved
@ -28,0 +42,4 @@
{{- if and (not $image_src) .Params.image }}
{{- $image_src = .Params.image }}
{{- with .Params.image_alt }}
{{- $image_alt = . }}
Collaborator

idem

idem
paspo marked this conversation as resolved
@ -28,0 +45,4 @@
{{- $image_alt = . }}
{{- end }}
{{- with .Params.image_caption }}
{{- $image_caption = . }}
Collaborator

idem

idem
paspo marked this conversation as resolved
@ -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 -}}/>
Collaborator

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.

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 `<?php if ($image.height*$ratio > $image.Width) echo "style=\"width: auto;\"" ?>) E comunque compattando l'espressione della condizione.
Author
Owner

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

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
paspo marked this conversation as resolved
@ -7,2 +7,3 @@
{{- $image_caption := "" }}
{{/* immagine di default */}}
{{- $featimg := (resources.Get "/images/placeholder_autore.png").RelPermalink -}}
{{- $image := (resources.Get "/images/placeholder_autore.png") -}}
Collaborator

idem

idem
paspo marked this conversation as resolved
@ -38,0 +12,4 @@
{{- if .Params.image_list }}
{{- $image_src = .Params.image_list.src }}
{{- with .Params.image_list.alt }}
{{- $image_alt = . }}
Collaborator

idem

idem
paspo marked this conversation as resolved
@ -38,0 +15,4 @@
{{- $image_alt = . }}
{{- end }}
{{- with .Params.image_list.caption }}
{{- $image_caption = . }}
Collaborator

idem

idem
paspo marked this conversation as resolved
@ -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 = . }}
Collaborator

idem

idem
paspo marked this conversation as resolved
@ -40,0 +34,4 @@
{{- if and (not $image_src) .Params.image }}
{{- $image_src = .Params.image }}
{{- with .Params.image_alt }}
{{- $image_alt = . }}
Collaborator

idem

idem
paspo marked this conversation as resolved
@ -40,0 +37,4 @@
{{- $image_alt = . }}
{{- end }}
{{- with .Params.image_caption }}
{{- $image_caption = . }}
Collaborator

idem

idem
paspo marked this conversation as resolved
@ -40,0 +43,4 @@
{{- if $image_src }}
{{- $image = .Page.Resources.GetMatch $image_src }}
{{- end}}
Collaborator

idem (fallback qui sotto)

idem (fallback qui sotto)
paspo marked this conversation as resolved
amreo canceled auto merging this pull request when all checks succeed 2024-11-12 12:10:21 +00:00
paspo force-pushed big-images from 1aacef7869 to 3e313a4cb8 2024-11-12 13:07:20 +00:00 Compare
paspo merged commit f2aa07712c into main 2024-11-12 13:08:18 +00:00
paspo deleted branch big-images 2024-11-12 13:08:18 +00:00
Author
Owner

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

secondo me ha rimosso la cosa quando gli ho esplicitato #342 come dipendenza

> 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 secondo me ha rimosso la cosa quando gli ho esplicitato #342 come dipendenza
Member

Giusto! Interessante funzionalità.

Giusto! Interessante funzionalità.
Sign in to join this conversation.
No Reviewers
3 Participants
Notifications
Due Date
No due date set.
Reference: BgLUG/bglug.it#340
No description provided.