Code review comment for lp://qastaging/~l3on/ubuntu-it-ask/uit-style

2012/9/24 Giuseppe Terrasi <email address hidden>:
> Wow!
>
> Leo, qui [¹] mi ero segnato una serie di modifiche che credo siano tutt'ora attuali e che forse è il caso di mantenere (vedo che anche questi file sono stati rimossi). Per chiarire cosa intendo consiglio un diff tra il file presente nella cartella light e in quello presente nella cartella default.
>
> 1) forum/skins/light/media/js/wmd/wmd.js: alcune stringhe sono presenti direttamente nei file javascript.
> 2) forum/skins/light/media/js/osqa.ask.js: stessa cosa

Io opterei per un approccio diverso.. ovvero la creazione di un js
locale per cambiare solo quelle stringhe (invece di mantenere copia
del il file originale).

Vedi il commit 76, funziona bene per il punto 1 e 2.

> 3) forum/skins/light/templates/notifications/digest.html: tradotte stringhe non presenti nel .po.

Qui posso farci poco... ma tanto, detto fra noi: chi legge le email
può anche passarci su... bisognerebbe però aprire un bug e segnalare
il problema.

> 4) forum/skins/light/templates/auth/signin.html: qui avevo nascosto ciò che adesso è visibile all'url account/signin/ cioè la possibilità di registrarsi inserendo un indirizzo email, costringendo così gli utenti ad usare solo openid.

Fatto con CSS, vedi commit 77.

> Non ne sono sicuro ma anche qui c'era qualcosa che può tornare utile per i problemi di internazionalizzazione che abbiamo:
>
> 5) forum/skins/light/templates/users/online_users.html
> 6) forum/skins/light/templates/users/subscriptions.html
> 7) forum/skins/light/templates/users/users.html

Non conosco la storia di questi file, ma sembrano differire di poco
dal tema 'default'. Non è che semplicemente vanno aggiornate le
traduzioni?

> 8) forum/templatetags/extra_tags.py
E qui ci posso fare davvero poco...

Comunque... che ne pensi?

Ciao!

Leo.

--
Ubuntu Member - http://launchpad.net/~l3on
Home Page - http://leoiannacone.com
GPG Key Id - 0xD282FC25