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

Widget - ordinamenti diversi #46

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Widget - ordinamenti diversi #46

wants to merge 3 commits into from

Conversation

leouzz
Copy link

@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
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
Member

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

@MarcoGargano
Copy link
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
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
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
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