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

Il giorno 25/set/2012 13:52, "Leo Iannacone" <email address hidden> ha scritto:
>
> 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.

Ottimo. Visto adesso i nuovi commit.

> > 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.

Su upstream non possiamo contarci. Noi il .po lo abbiamo mandato e bug ne
abbiamo gia' aperti tutto tace... Non me ne preoccupo di questo punto tanto
c'e' lo segnaleranno gli utenti e al quel punto lo traccero' nel bug
tracker e nel caso risolvero' allo stesso modo.

> > 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.

Ottimo anche qui. Era il risultato che mi interessava non che fosse stata
tolta la revisione precedente.

> > 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?

Qui mi propongo di riguardarci per verificare a cosa servissero.

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

Come prima. Ma se serve a patchare qualosa sono per rimetterlo.
L'ultimo lavoro dovrebbe essere poi quello di riaggiornare il file FIXME.

>
>
> Comunque... che ne pensi?
>
> Ciao!
>
> Leo.

Ne penso benissimo!
A tra poco.
Giuseppe