search

retepasw / joomla-italiapa

Implementazione delle Linee guida di design per i servizi web della PA basato su it Web Toolkit.
https://www.eshiol.it/it/b013
GNU General Public License v3.0
13 stars 9 forks source link

Rimosso caricamento font da fonts.googleapis.com #473

Closed eshiol closed 2 years ago

eshiol commented 2 years ago

Summary of Changes

Rimosso caricamento font da fonts.googleapis.com.

Testing Instructions

Expected result

Actual result

Documentation Changes Required

sammyildruido commented 2 years ago

Ancora risultano collegamenti con google, praticamente non toglie i riferimenti su Index.php ed offline.php:

queste le righe dopo aver applicato la patch:

index.php

89 <link href='//fonts.googleapis.com/css?family=Titillium+Web:400,400italic,700,' rel='stylesheet' type='text/css' />
90 <link href="https://fonts.googleapis.com/icon?family=Material+Icons" rel="stylesheet">
91

offline.php

98 <link href='//fonts.googleapis.com/css?family=Titillium+Web:400,400italic,700,' rel='stylesheet' type='text/css' />
99 <link href="https://fonts.googleapis.com/icon?family=Material+Icons" rel="stylesheet">
100

praticamente non vengono tolte. :-)

togliendo queste righe a mano sembra che funzioni (ho provato a commentarle)

eshiol commented 2 years ago

Credo tu abbia commesso qualche errore nell'applicazione della patch. Hai usato come riferimento questa guida? ItaliaPA Bug Test - Guida per principianti

sammyildruido commented 2 years ago

Ho usato la tua guida. L'unico problema è che ho usato la versione del template ultima dopo aver messo la pacth. Forse per quello?

Il mar 9 ago 2022, 22:20 Helios Ciancio @.***> ha scritto:

Credo tu abbia commesso qualche errore nell'applicazione della patch. Hai usato come riferimento questa guida? ItaliaPA Bug Test - Guida per principianti http://www.eshiol.it/it/template-italiapa/guide/template/bug-test.html

— Reply to this email directly, view it on GitHub https://github.com/retepasw/joomla-italiapa/pull/473#issuecomment-1209833566, or unsubscribe https://github.com/notifications/unsubscribe-auth/AF7VF3KX7WJTXNTX57I5JT3VYK4PFANCNFSM556ODJKA . You are receiving this because you commented.Message ID: @.***>

eshiol commented 2 years ago

Scusate la mia grande ignoranza ... ma, cortesemente, potreste indicarmi dove lo trovo il file della patch e come installarlo? Grazie

Ci sono diversi modi per testare le PR. Puoi scaricare i file direttamente da github oppore seguire la guida ItaliaPA Bug Test - Guida per principianti

eshiol commented 2 years ago

Ho usato la tua guida. L'unico problema è che ho usato la versione del template ultima dopo aver messo la pacth. Forse per quello?

Probabile. Prima va installata l'ultima nightly build e poi vanno scaricate le PR da testare.

giuoll commented 2 years ago

Screenshot (5)

Buongiorno

vittorioalfieri commented 2 years ago

Confermo anche io che dopo l'applicazione della patch nella index.php rimangono due riferimenti a google fonts: `

` prima della patch avevo il template: italiapa-3.9.0.21_mod

operazioni eseguite: 1) installato componente Patch Tester 3.0.0 2) installato template ItaliaPA Night build 3.9.0.22-dev 3) applicata la patch in modo coretto (allego screen)

Joomla! Patch Tester - Amministrazione

Aggiornamento: rimosse manualmente le due righe da index.php (righe 18 e 19 per me) il sito sembra funzionare correttamente.

giuoll commented 2 years ago

Confermo anche io che dopo l'applicazione della patch nella index.php rimangono due riferimenti a google fonts: <link href='//fonts.googleapis.com/css?family=Titillium+Web:400,400italic,700,' rel='stylesheet' type='text/css' /> <link href="https://fonts.googleapis.com/icon?family=Material+Icons" rel="stylesheet"> prima della patch avevo il template: italiapa-3.9.0.21_mod

operazioni eseguite:

  1. installato componente Patch Tester 3.0.0
  2. installato template ItaliaPA Night build 3.9.0.22-dev
  3. applicata la patch in modo coretto (allego screen)

Joomla! Patch Tester - Amministrazione

Aggiornamento: rimosse manualmente le due righe da index.php (righe 18 e 19 per me) il sito sembra funzionare correttamente.

Da me non sono presenti riferimenti a google fonts

eshiol commented 2 years ago

Davvero strano. I riferimenti a google fonts dovrebbero essere stati tolti ovunque.

Stranamente Patch Tester non carica questi due file index.php offline.php

mentre carica senza problemi component.php error.php

L'alternativa a Patch Tester è scaricare manualmente i singoli file.

vittorioalfieri commented 2 years ago

Potrebbe essere che incautamente io abbia aperto index.php dal menu dei template (funzione di modifica file) modificandone data e ora e quindi quando ho applicato il night build template questi abbia saltato il file index.php ... che quindi per joomla risulterebbe "bloccato" perché modificato a mano.... ?

comunque ora nel mio sito di test sembra funzionare tutto, al limite via ftp mi scarico la cartella del template e la ricarico sul sito di produzione... o pensavi di rilasciare una mini release ?

AntonioGanci65 commented 2 years ago

Ho due domanda da fare: 1) Dove posso trovare template ItaliaPA Night build 3.9.0.22-dev - o visto che sul sito trovo solo la versione template ItaliaPA Night build 3.9.0.21-dev di gennaio 2022 2) posso installare la versione Patch Tester 4.2.1 Release invece dalla versione componente Patch Tester 3.0.0 GRAZIE DELLE INFO - Antonio Ganci

angiesg79 commented 2 years ago

Ho caricato i file tramite ftp e confermo il funzionamento. Ho lavorato su questo sito https://www.icvenosa.edu.it e ho fatto le verifiche qui https://webbkoll.dataskydd.net/en/

vittorioalfieri commented 2 years ago

1) puoi trovare la night build alla pagina http://www.eshiol.it/it/template-italiapa/nightly-builds.html

2) no Patch Tester 4.X.X è fatto solo per Joomla 4 e quindi impossibile da installare su J3

martiros1966 commented 2 years ago

Scusate la mi ignoranza; ma come faccio a scaricare direttamente i file da sovrascrivere per abolire lo scaricamento dei font google nel mio sito con template ITALIAPA? Io riesco solo a vedere i file interessati ma non a scaricarli.

alessandro40ch commented 2 years ago

Patch applicata e perfettamente funzionante. Caricamento font da fonts.googleapis.com rimosso completamente

MarcelloRocca commented 2 years ago

Ho seguito la la guida ItaliaPA Bug Test - Guida per principianti e installato la versione patch tester 3.0.0. Dopo aver cliccato su Preleva i dati mi appare un elenco di 205 titoli, ma non riesco a capire quale sia quello relativo a fonts.googleapis.com.

vittorioalfieri commented 2 years ago

Ho seguito la la guida ItaliaPA Bug Test - Guida per principianti e installato la versione patch tester 3.0.0. Dopo aver cliccato su Preleva i dati mi appare un elenco di 205 titoli, ma non riesco a capire quale sia quello relativo a fonts.googleapis.com.

devi configurare patch tester come scritto sulla guida, nelle opzioni devi puntare al repository di italiapa

MarcelloRocca commented 2 years ago

Ho seguito la guida ma il Passaggio 4. - preleva le correzioni presenta una situazione diversa da quella presente nella versione patch tester 3.0.0. che utilizzo io. Ti allego due immagini dove ho inserito le mie credenziali di accesso a Github.com. Immagine 1 Immagine 2 Dove sbaglio?

vittorioalfieri commented 2 years ago

Ho seguito la guida ma il Passaggio 4. - preleva le correzioni presenta una situazione diversa da quella presente nella versione patch tester 3.0.0. che utilizzo io. Ti allego due immagini dove ho inserito le mie credenziali di accesso a Github.com. Immagine 1 Immagine 2 Dove sbaglio?

image

le credenziali se devi fare solo 1 o 2 download non servono

jofaner commented 2 years ago

Ho sostituito manualmente il template via ftp. Sembra funzionare tutto. grazie

martiros1966 commented 2 years ago

Salve, vorrei segnalare ciò che ho fatto e il relativo risultato: 1) ho scaricato i file del template corretti forniti da Helios 2) ho sovrascritto i vecchi file con i nuovi via ftp 3) dopo aver cancellato la cache del browser il sito sembra funzionare correttamente 4) tuttavia ho controllato il sito con lo strumento https://webbkoll.dataskydd.net/en e mi ritrovo ancora dei riferimenti di google font nei seguenti file: templates\italiapa\build\0.chunk.js e templates\italiapa\build\0.chunk.js.map

Non so se ho sbagliato qualcosa ma sicuramente sembra che io NON abbia risolto il problema al 100% C'è qualcuno che può darmi un suggerimento in merito?

martiros1966 commented 2 years ago

scusate ho dimenticato a dare il link al mio sito: https://www.alberghieroerice.edu.it/

AntonioGanci65 commented 2 years ago

Salve martiros66 sono Antonio Ganci un tuo collega di Alcamo, vorrei sapere ma tu hai solo sovrascritto i file del template via FTP, non hai usato la installato componente Patch Tester 3.0.0.

martiros1966 commented 2 years ago

Salve martiros66 sono Antonio Ganci un tuo collega di Alcamo, vorrei sapere ma tu hai solo sovrascritto i file del template via FTP, non hai usato la installato componente Patch Tester 3.0.0.

Ciao Antonio... Ho scaricato i file della cartella "italiapa" e ho sovrascritto i file via ftp. Tuttavia facendo un controllo con il seguente sito: https://webbkoll.dataskydd.net/en rimangono 5 chiamate ad un font di google chiamato "Roboto". Tali chiamate dovrebbero trovarsi nei seguenti files: templates\italiapa\build\0.chunk.js e templates\italiapa\build\0.chunk.js.map Per il resto tutto funziona correttamente. Spero che qualcuno mi possa aiutare per il suesposto problema.

MarcelloRocca commented 2 years ago

Aggiornando il template alla versione 3.10 ho risolto tutti i problemi.Inviato dal mio dispositivo Huawei-------- Messaggio originale --------Oggetto: Re: [retepasw/joomla-italiapa] Rimosso caricamento font da fonts.googleapis.com (PR #473)Da: martiros1966 A: retepasw/joomla-italiapa CC: MarcelloRocca ,Comment

Salve martiros66 sono Antonio Ganci un tuo collega di Alcamo, vorrei sapere ma tu hai solo sovrascritto i file del template via FTP, non hai usato la installato componente Patch Tester 3.0.0.

Ciao Antonio... Ho scaricato i file della cartella "italiapa" e ho sovrascritto i file via ftp. Tuttavia facendo un controllo con il seguente sito: https://webbkoll.dataskydd.net/en rimangono 5 chiamate ad un font di google chiamato "Roboto". Tali chiamate dovrebbero trovarsi nei seguenti files: templates\italiapa\build\0.chunk.js e templates\italiapa\build\0.chunk.js.map Per il resto tutto funziona correttamente. Spero che qualcuno mi possa aiutare per il suesposto problema.

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.> [ { @.": "http://schema.org", @.": "EmailMessage", "potentialAction": { @.": "ViewAction", "target": "https://github.com/retepasw/joomla-italiapa/pull/473#issuecomment-1212262049", "url": "https://github.com/retepasw/joomla-italiapa/pull/473#issuecomment-1212262049", "name": "View Pull Request" }, "description": "View this Pull Request on GitHub", "publisher": { @.***": "Organization", "name": "GitHub", "url": "https://github.com" } } ]

luckybla commented 2 years ago

Buongiorno, una domanda. Dopo aver eseguito i seguenti passi: 1) installato template ItaliaPA Night build 3.9.0.22-dev 2) installato componente Patch Tester 3.0.0 se come Repository GitHub seleziono Joomla! CMS API il prelievo dei dati mi produce 208 risultati, mentre se inserisco

eshiol commented 2 years ago

La PR è stata chiusa ed è stato fatto il merge. Non è più possibile testarla tramite Patch tester. La patch è stata resa disponibile nella nuova versione del template ItaliaPA 3.10.

jofaner commented 2 years ago

Buongiorno, come dicevo ho aggiornato sostituendo manualmente il template via ftp. Analizzando con webbkoll compaiono tuttavia ancora riferimenti a fonts.googleapis. Ammesso che sia un problema, cosa resterebbe da fare? Grazie! screen

luckybla commented 2 years ago

Problema risolto, grazie mille Helios

Il giorno ven 12 ago 2022 alle ore 11:13 jofaner @.***> ha scritto:

Buongiorno, come dicevo ho aggiornato sostituendo manualmente il template via ftp. Analizzando con webbkoll compaiono tuttavia ancora riferimenti a fonts.googleapis. Cosa resta da fare? Grazie [image: screen] https://user-images.githubusercontent.com/60347917/184323653-464b0efd-02da-4b67-b351-d32bebbeca32.jpg

— Reply to this email directly, view it on GitHub https://github.com/retepasw/joomla-italiapa/pull/473#issuecomment-1212900758, or unsubscribe https://github.com/notifications/unsubscribe-auth/A2OURUIZHB3USFCHSZE7IVTVYYIVNANCNFSM556ODJKA . You are receiving this because you commented.Message ID: @.***>

AntonioGanci65 commented 2 years ago

Salve vorrei ho fatto la procedura nel sito www.icnavarra.edu.it ed è risultato tutto a posto tranne per un altyro font che non riesco a risolvere: fonts.gstatic.com 142.250.186.163 https://fonts.gstatic.com/s/iceland/v9/rax9HiuFsdMNOnWPaKtMBA.ttf quancuno mi sà dire come togliere questo collegamento? Grazie in anticipo.

eshiol commented 2 years ago

Basta aggiornare il template alla versione 3.10

jofaner commented 2 years ago

Buongiorno Helios, ho aggiornato il template prelevando i files con le tue modifiche e sostituendolo via ftp. Ho fatto questo appena prima che venisse rilasciato il pacchetto 3.10. Comunque non ho sovrascritto, ho cancellato la cartella italiapa e l'ho sostituita con la nuova. Passando il sito in webbkroll permangono tuttavia alcuni riferimenti a google fonts: fonts.googleapis.com https://fonts.googleapis.com/css?family=Roboto:300,400,500,700 fonts.gstatic.com https://fonts.gstatic.com/s/roboto/v30/KFOlCnqEu92Fr1MmEU9fBBc4.woff2 https://fonts.gstatic.com/s/roboto/v30/KFOmCnqEu92Fr1Mu4mxK.woff2

A questo punto è possibile eliminarli manualmente o meglio provare ad installare il pacchetto completo? grazie in anticipo Giovanni

sito della mia scuola: https://www.liceofracastoro.edu.it/

eshiol commented 2 years ago

Per supporto su problemi legati ai singoli siti web vi invito ad utilizzare i canali web https://www.facebook.com/groups/pasw4joomla https://www.paswjoomla.net/joomla/forum/template-italiapa.html https://forum.joomla.it/index.php/board,97.0.html