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

Revision history for this message
Giuseppe Terrasi (giuseppeterrasi-deactivatedaccount) wrote :

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
3) forum/skins/light/templates/notifications/digest.html: tradotte stringhe non presenti nel .po.
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.

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
8) forum/templatetags/extra_tags.py

Ciao!

[1] http://bazaar.launchpad.net/~ubuntu-it-ask/ubuntu-it-ask/dev/view/head:/00-FIXME

review: Needs Fixing

« Back to merge proposal