Merge lp:~giuseppeterrasi-deactivatedaccount/ubuntu-it-ask/subscriptions.html into lp:~ubuntu-it-ask/ubuntu-it-ask/dev

Proposed by Giuseppe Terrasi
Status: Merged
Merged at revision: 57
Proposed branch: lp:~giuseppeterrasi-deactivatedaccount/ubuntu-it-ask/subscriptions.html
Merge into: lp:~ubuntu-it-ask/ubuntu-it-ask/dev
Diff against target: 69 lines (+22/-1)
4 files modified
00-FIXME (+8/-0)
forum/skins/light/media/js/ask.jquery.header.string.replace.js (+1/-1)
forum/skins/light/media/js/ask.jquery.subscriptions.string.replace.js (+10/-0)
forum/skins/light/templates/users/subscriptions.html (+3/-0)
To merge this branch: bzr merge lp:~giuseppeterrasi-deactivatedaccount/ubuntu-it-ask/subscriptions.html
Reviewer Review Type Date Requested Status
Ubuntu-it Ask Pending
Review via email: mp+118238@code.launchpad.net

Description of the change

In contesti con link dinamici non posso usare replaceWith di jQuery.

Con questo semplice javascript risolvo problema creato dal file .po upstream che in questa pagina

users/1/nomeutente/subscriptions/

rende incoerenti le traduzioni che rispettivamente sono "Impostazioni" e "gestisci".

Il javascript rende minuscolo "impostazioni".

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file '00-FIXME'
2--- 00-FIXME 2012-08-03 19:06:00 +0000
3+++ 00-FIXME 2012-08-04 06:36:19 +0000
4@@ -39,6 +39,12 @@
5 Questi javascript usando il metodo replaceWith agiscono sui selettori css presenti nell'output html ed eseguono una serie di sostituzioni.
6 Nello specifico, l'introduzione di questo work-around serve a risolvere il problema della capitalizzazone del testo mostrato nell'header che risulta essere difficoltoso per via delle diverse stringhe del file .po comuni a più parti dell'interfaccia.
7
8+= ask.jquery.subscriptions.string.replace.js =
9+
10+Eliminata capitalizzazione parola "impostazioni" in subscriptions.html usando javascript. La presenza di un link dinamico non permette l'uso di string replace. L'effetto è stato raggiunto in questa maniera:
11+
12+var subscriptionDiv = $('#navBar a#subscriptions_settings_button');
13+subscriptionDiv.text('impostazioni');
14
15 = forum/skins/light/templates/users/online_users.html =
16 FIXME: proporre cambiamento upstream come da lp:~giuseppeterrasi/ubuntu-it-ask/l18ned
17@@ -51,3 +57,5 @@
18
19 = forum/templatetags/extra_tags.py =
20 FIXME: proporre cambiamento upstream come da lp:~giuseppeterrasi/ubuntu-it-ask/l18ned
21+
22+
23
24=== modified file 'forum/skins/light/media/js/ask.jquery.header.string.replace.js'
25--- forum/skins/light/media/js/ask.jquery.header.string.replace.js 2012-08-02 18:27:03 +0000
26+++ forum/skins/light/media/js/ask.jquery.header.string.replace.js 2012-08-04 06:36:19 +0000
27@@ -1,5 +1,5 @@
28 /*
29- * Title: jQuery string replace
30+ * Title: jQuery header string replace
31 * Description: customize header link in order to capitalize first letter using jquery replaceWith method.
32 *
33 * Copyright (c) 2012 Ubuntu-it Ask Team - https://launchpad.net/~ubuntu-it-ask
34
35=== added file 'forum/skins/light/media/js/ask.jquery.subscriptions.string.replace.js'
36--- forum/skins/light/media/js/ask.jquery.subscriptions.string.replace.js 1970-01-01 00:00:00 +0000
37+++ forum/skins/light/media/js/ask.jquery.subscriptions.string.replace.js 2012-08-04 06:36:19 +0000
38@@ -0,0 +1,10 @@
39+/*
40+ * Title: jQuery subscriptions string replace
41+ * Description: remove capitalization in "impostazioni" word using javascript.
42+ *
43+ * Copyright (c) 2012 Ubuntu-it Ask Team - https://launchpad.net/~ubuntu-it-ask
44+ * GNU GPL 3 LICENSE
45+ */
46+
47+var subscriptionDiv = $('#navBar a#subscriptions_settings_button');
48+subscriptionDiv.text('impostazioni');
49
50=== modified file 'forum/skins/light/templates/users/subscriptions.html'
51--- forum/skins/light/templates/users/subscriptions.html 2012-08-01 19:54:12 +0000
52+++ forum/skins/light/templates/users/subscriptions.html 2012-08-04 06:36:19 +0000
53@@ -26,6 +26,7 @@
54 //location.reload(true);
55 }
56 </script>
57+
58 {% endblock %}
59
60 {% block usercontent %}
61@@ -36,6 +37,8 @@
62 <a href='{{ view_user.get_user_subscriptions_url }}?tab=manage' id="subscriptions_management_button" {% if manage_open %}class="on"{% endif %}>{% trans "manage" %}</a>
63 </div></div>
64
65+<script type="text/javascript" src="/m/light/media/js/ask.jquery.subscriptions.string.replace.js"></script>
66+
67 <br/>
68
69 {% if not manage_open %}

Subscribers

No one subscribed via source and target branches