Merge lp:~cjohnston/loco-team-portal/706657 into lp:loco-team-portal

Proposed by Chris Johnston
Status: Merged
Approved by: Chris Johnston
Approved revision: 385
Merged at revision: 386
Proposed branch: lp:~cjohnston/loco-team-portal/706657
Merge into: lp:loco-team-portal
Diff against target: 91 lines (+34/-11)
5 files modified
loco_directory/media/css/newstyle.css (+12/-0)
loco_directory/templates/irc_chat.html (+1/-1)
loco_directory/templates/meetings/team_meeting_detail.inc.html (+11/-0)
loco_directory/templates/meetings/team_meeting_detail_basic.inc.html (+1/-1)
loco_directory/templates/teams/team_detail.html (+9/-9)
To merge this branch: bzr merge lp:~cjohnston/loco-team-portal/706657
Reviewer Review Type Date Requested Status
Nigel Babu (community) Approve
LoCo Team Portal Developers Pending
Review via email: mp+48413@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Nigel Babu (nigelbabu) wrote :

We've been doing inline style to position the webchat div from last release. Since we're doing that almost everywhere (and its my mistake), we should probably move all of that into the css file and give it a class. Looks fine otherwise.

review: Needs Fixing
385. By Chris Johnston

Changes style from inline to stylesheet.

Revision history for this message
Nigel Babu (nigelbabu) wrote :

Looks good :)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'loco_directory/media/css/newstyle.css'
2--- loco_directory/media/css/newstyle.css 2011-01-23 14:48:17 +0000
3+++ loco_directory/media/css/newstyle.css 2011-02-03 01:46:26 +0000
4@@ -460,3 +460,15 @@
5 margin-left: 30px;
6 font-size: 0.9em;
7 }
8+
9+#webchat {
10+ display: none;
11+ width: 900px;
12+ margin: 0 auto;
13+}
14+
15+#ircchat {
16+ display: block;
17+ width: 900px;
18+ margin: 0 auto;
19+}
20
21=== modified file 'loco_directory/templates/irc_chat.html'
22--- loco_directory/templates/irc_chat.html 2011-01-25 02:14:26 +0000
23+++ loco_directory/templates/irc_chat.html 2011-02-03 01:46:26 +0000
24@@ -15,7 +15,7 @@
25 <article class="main-content alone">
26 <h2>Webchat</h2>
27 <br />
28- <div style="display: block; width: 900px; margin: 0 auto;">
29+ <div id="ircchat">
30 {% if user.is_authenticated %}
31 <iframe src="http://webchat.freenode.net/?channels=#ubuntu-locoteams&prompt=1&uio=MTE9MzE28&nick={{ user.username }}" width="900" height="400"></iframe>
32 {% else %}
33
34=== modified file 'loco_directory/templates/meetings/team_meeting_detail.inc.html'
35--- loco_directory/templates/meetings/team_meeting_detail.inc.html 2010-12-09 15:17:27 +0000
36+++ loco_directory/templates/meetings/team_meeting_detail.inc.html 2011-02-03 01:46:26 +0000
37@@ -11,3 +11,14 @@
38 {% include "meetings/team_meeting_detail_agenda.inc.html" %}
39 </article>
40
41+<article class="main-content">
42+ {% block general %}
43+ <div id="webchat">
44+ {% if user.is_authenticated %}
45+ <iframe src="http://webchat.freenode.net/?channels={{ team_meeting_object.channel }}&prompt=1&uio=OT10cnVlJjEwPXRydWUmMTE9MjM218&nick={{ user.username }}" width="900" height="400"></iframe>
46+ {% else %}
47+ <iframe src="http://webchat.freenode.net/?channels={{ team_meeting_object.channel }}&prompt=1&uio=OT10cnVlJjEwPXRydWUmMTE9MjM218&nick=locodir-user" width="900" height="400"></iframe>
48+ {% endif %}
49+ </div>
50+ {% endblock %}
51+</article>
52
53=== modified file 'loco_directory/templates/meetings/team_meeting_detail_basic.inc.html'
54--- loco_directory/templates/meetings/team_meeting_detail_basic.inc.html 2011-01-24 02:44:41 +0000
55+++ loco_directory/templates/meetings/team_meeting_detail_basic.inc.html 2011-02-03 01:46:26 +0000
56@@ -21,7 +21,7 @@
57 </tr>
58 <tr>
59 <th class="form-item-label" scope="row"><label>{% trans "Meeting Channel:" %}</label></th>
60- <td class="form-item-value">{{ team_meeting_object.channel }}</td>
61+ <td class="form-item-value"><a title="{% trans "Join Channel" %} {{ team_meeting_object.channel }}" href="#" onclick="$('#webchat').show('fast', function() {$(window).scrollTop($('#webchat').position().top);} );">{{ team_meeting_object.channel }}</a></td>
62 </tr>
63 <tr>
64 <th class="form-item-label" scope="row"><label>{% trans "Meeting Chair:" %}</label></th>
65
66=== modified file 'loco_directory/templates/teams/team_detail.html'
67--- loco_directory/templates/teams/team_detail.html 2011-01-26 00:57:42 +0000
68+++ loco_directory/templates/teams/team_detail.html 2011-02-03 01:46:26 +0000
69@@ -245,13 +245,13 @@
70
71 {% endblock %}
72 {% block general %}
73-{% if team.irc_chan %}
74- <div id="webchat" style="display: none; width: 900px; margin: 0 auto;">
75- {% if user.is_authenticated %}
76- <iframe src="http://webchat.freenode.net/?channels={{ team.irc_chan }}&prompt=1&uio=OT10cnVlJjEwPXRydWUmMTE9MjM218&nick={{ user.username }}" width="900" height="400"></iframe>
77- {% else %}
78- <iframe src="http://webchat.freenode.net/?channels={{ team.irc_chan }}&prompt=1&uio=OT10cnVlJjEwPXRydWUmMTE9MjM218&nick=locodir-user" width="900" height="400"></iframe>
79- {% endif %}
80-</div>
81-{% endif %}
82+ {% if team.irc_chan %}
83+ <div id="webchat">
84+ {% if user.is_authenticated %}
85+ <iframe src="http://webchat.freenode.net/?channels={{ team.irc_chan }}&prompt=1&uio=OT10cnVlJjEwPXRydWUmMTE9MjM218&nick={{ user.username }}" width="900" height="400"></iframe>
86+ {% else %}
87+ <iframe src="http://webchat.freenode.net/?channels={{ team.irc_chan }}&prompt=1&uio=OT10cnVlJjEwPXRydWUmMTE9MjM218&nick=locodir-user" width="900" height="400"></iframe>
88+ {% endif %}
89+ </div>
90+ {% endif %}
91 {% endblock %}

Subscribers

People subscribed via source and target branches