Corregge errore di formattazione sui box

italia / design-linee-guida-docs

Creative Commons Attribution 4.0 International

12 stars 19 forks source link

Corregge errore di formattazione sui box #6

Closed francescomda closed 5 years ago

francescomda commented 5 years ago

Corregge gli errori di formattazione dei box nel capitolo "Linguaggio" delle linee guida. Nell'approvare la PR, attenzione al box "Archivi di contenuti multimediali online", perché non riesco a vedere la modifica fatta in preview.

alterat commented 5 years ago

Sono d'accordo! Ho già le modifiche pronte, appena @francescomda mi dà i permessi per lavorare sul suo fork le inserisco direttamente in questa PR.

francescozaia commented 5 years ago

Ho provato la modifica in locale e non mi pare sistemi la formattazione del capitolo "Linguaggio". Io lo vedo così:

Purtroppo ho problemi a fare modifiche direttamente in quel branch per sistemare. Voi come avete verificato il corretto funzionamento?

alterat commented 5 years ago

Ciao @francescozaia, grazie della revisione! In effetti ci sono dei problemi di formattazione con il nuovo tema, ora faccio le correzioni necessarie...

francescozaia commented 5 years ago

Se vuoi @atorin, ho appena aggiornato il repo sistemando riferimenti al tema ufficiale ed altro, ed aggiunto un minimo di documentazione al README.md. È tutto su master ed ora dovrebbe essere più facile testare tutto in locale.

alterat commented 5 years ago

Ho corretto l'errore che impediva ai box di approfondimento di essere generati. Era necessario aggiungere una linea vuota prima del titoletto del box, per evitare che questo venisse interpretato come un parametro.

C'è però un altro serio bug che credo abbia a che fare con il tema. Tutto il testo dei paragrafi all'interno dei vari box Approfondimento è stato messo nel primo box, lasciando nei successivi soltanto gli elenchi puntati.

francescozaia commented 5 years ago

Duplicata da #9

pdavide commented 5 years ago

Il bug relativo ai box di approfondimento è stato fixato.

alterat commented 5 years ago

Grazie @pdavide per il fix... Io però nella mia versione locale vedo questo per il primo box:

Il primo elenco dovrebbe essere scambiato con il secondo paragrafo. Altri box più sotto hanno hanno un problema simile.

Ho fatto pip install --upgrade git+https://github.com/italia/docs-italia-theme.git@bootstrap-italia prima di compilare nuovamente il doc. Magari c'è qualcosa che non va dal mio lato...

@francescozaia puoi confermare o meno questo comportamento con la tua versione locale, magari direttamente quella della ---#6--- volevo dire #9 ?

pdavide commented 5 years ago

@atorin c'era un ulteriore bug perché lo script prendeva in considerazione solo i paragrafi e non gli elenchi. Problema risolto alla radice con https://github.com/italia/docs-italia-theme/pull/162.

alterat commented 5 years ago

Confermo, il nuovo merge ha risolto il problema! Grazie!