Merge lp://staging/~rpadovani/ubuntu-it-ask/users into lp://staging/ubuntu-it-ask

Proposed by Riccardo Padovani
Status: Merged
Approved by: Giuseppe Terrasi
Approved revision: 33
Merged at revision: 34
Proposed branch: lp://staging/~rpadovani/ubuntu-it-ask/users
Merge into: lp://staging/ubuntu-it-ask
Diff against target: 225 lines (+30/-24)
5 files modified
forum/skins/light/media/style/style.css (+14/-12)
forum/skins/light/media/style/user.css (+2/-2)
forum/skins/light/templates/logout.html (+3/-1)
forum/skins/light/templates/users/edit.html (+7/-5)
forum/skins/light/templates/users/info.html (+4/-4)
To merge this branch: bzr merge lp://staging/~rpadovani/ubuntu-it-ask/users
Reviewer Review Type Date Requested Status
Giuseppe Terrasi (community) approvato Approve
Review via email: mp+114144@code.staging.launchpad.net

Commit message

Merge 114144 into trunk, thanks to Riccardo Padovani!

Description of the change

- New style for logout, profile and users page;
- Minor changes.

To post a comment you must log in.
Revision history for this message
Giuseppe Terrasi (giuseppeterrasi-deactivatedaccount) wrote :

Approvato!
Grazie Riccardo!

review: Approve (approvato)
Revision history for this message
Milo Casagrande (milo) wrote :

Una nota su questo.

On Tue, Jul 10, 2012 at 11:50 AM, Riccardo Padovani
<email address hidden> wrote:
> + <div id="left" style="float:left;width:15%;text-align:center;">
> {% if user.email %}
> {% gravatar user 128 %}
> {% else %}
> <img src="{% media "/media/images/nophoto.png" %}">
> {% endif %}
> - <div style="padding:20px 0 0 20px;font-weight:bold;font-size:150%">
> + <div style="padding:10px 0 0 0;font-weight:bold;font-size:150%">
> <a href="http://www.gravatar.com/" target="_blank"
> title="gravatar {% trans "image associated with your email address" %}">{% trans "gravatar" %}</a>

In particolare qui sopra. Quella parte "traducibile" non è che sia il massimo.
In questo caso la parte realmente traducibile è:
"image associated..."

ma il testo finale, integro, è:
gravatar image associated...

Ora, in italiano la parte traducibile verrebbe fuori come:
immagine associata...

Completa, sarebbe:
gravatar immagine associata...

Che non si può vedere.
Se correggiamo upstream, ammesso che venga presa la modifica, poi è
necessario fare dei merge spaventosi per tenerci aggiornati (ci stavo
pensando l'altro giorno).
In ogni caso va sistemata, anche solo nel nostro tema, ma poi vanno
ricreati i file per le traduzioni con le nuove stringhe.

Ciao.

--
Milo Casagrande <email address hidden>

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches