Skip to content
This repository was archived by the owner on Jun 8, 2023. It is now read-only.

Widget - ordinamenti diversi#46

Open
leouzz wants to merge 3 commits into
italia:masterfrom
leouzz:patch-1
Open

Widget - ordinamenti diversi#46
leouzz wants to merge 3 commits into
italia:masterfrom
leouzz:patch-1

Conversation

@leouzz
Copy link
Copy Markdown

@leouzz leouzz commented Apr 23, 2020

Con la mia modifica è possibile scegliere tra diversi tipi di ordinamenti all'interno dei widget "Articoli a griglia" e "Articoli in Orizzontale"
-Random: 'rand'
-Per data di pubblicazione: 'publish_date'
-ordine alfabetico; 'title' 'ASC'
-per id (quello che prima era il non random)

Con la mia modifica è possibile scegliere tra diversi tipi di ordinamenti all'interno dei widget "Articoli a griglia" e "Articoli in Orizzontale"
-Random: 'rand'
-Per data di pubblicazione: 'publish_date'
-ordine alfabetico; 'title' 'ASC' 
-per id (quello che prima era il non random)
@MarcoGargano
Copy link
Copy Markdown
Member

@leouzz grazie mille!!! 🤩
L'ho provato al volo e sembra funzionare tutto correttamente... fantastico!
Faccio ancora un paio di test e lo inserisco nella prossima release.
Ottimo suggerimento e code, grazie ancora!

@MarcoGargano
Copy link
Copy Markdown
Member

@leouzz ho un timore 🤔
La modifica a $random così come l'hai fatta comprometterebbe i widget già in uso.

@MarcoGargano
Copy link
Copy Markdown
Member

Te lo confermo, si rompono tutti i widget già presenti.
Avresti un suggerimento per evitare che chi sta usando il tema e widget gli dia errore.

La tua proposta mi piace molto ma devo capire come gestire le installazioni già esistenti.

Mantenuti nomi variabili per mantenere i setting fatti nelle vecchie versioni.
aggiunte opzioni che fanno vedere correttamente nell'input radio se l'utente aveva lasciato l'ordinamento per id in una vecchia versione.
@leouzz
Copy link
Copy Markdown
Author

leouzz commented Apr 25, 2020

Grazie, dovrei aver sistemato, ho riutilizzato li stessi nomi variabile e fixato il menu che ora mostra la scelta fatta nella versione precedente.

@MarcoGargano
Copy link
Copy Markdown
Member

Grazie mille per la fix, ora non sembra più avere lo stesso problema di retrocompatibilità.
Lo testo ancora un po' e poi lo inserisco nella prossima release.
Grazie ancora 😃

@MarcoGargano
Copy link
Copy Markdown
Member

Ciao @leouzz
ho rilasciato una nuova versione (v 0.33) con il tuo contributo.
Non mi sono arrivate segnalazioni quindi spero sia andato tutto bene.
Se anche per te è andato a buon fine chiudo questa request.
Grazie ancora!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants