Description:
Merge branch 'master' into codejom (bug fix)
Commit status:
[Not Reviewed]
References:
merge default
Comments:
0 Commit comments 0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
Add another comment

r227:eda4ffd09089 - - 2 files changed: 14 inserted, 10 deleted

@@ -1,15 +1,14
1 1 - content_for :head do
2 2 = javascript_include_tag :defaults
3 - %script{:type => 'text/javascript', :src => '/javascripts/announcement_refresh.js'}
3 + = javascript_include_tag 'announcement_refresh.js'
4 4
5 5 = user_title_bar(@user)
6 6
7 - - if @announcements.length!=0
8 - .announcementbox
9 - %span{:class => 'title'}
10 - Announcements
11 - #announcementbox-body
12 - = render :partial => 'announcement', :collection => @announcements
7 + .announcementbox{:style => (@announcements.length==0 ? "display:none" : "")}
8 + %span{:class => 'title'}
9 + Announcements
10 + #announcementbox-body
11 + = render :partial => 'announcement', :collection => @announcements
13 12
14 13 %hr/
15 14
@@ -18,8 +18,13
18 18 method: 'get',
19 19 parameters: { recent: Announcement.mostRecentId },
20 20 onSuccess: function(transport) {
21 - var announcementBody = $("announcementbox-body");
22 - announcementBody.insert({ top: transport.responseText });
21 + if(transport.responseText.match(/\S/)!=null) {
22 + var announcementBody = $("announcementbox-body");
23 + announcementBody.insert({ top: transport.responseText });
24 + var announcementBoxes = $$(".announcementbox");
25 + if(announcementBoxes.length!=0)
26 + announcementBoxes[0].show();
27 + }
23 28 }
24 29 });
25 30 Announcement.registerRefreshEventTimer();
@@ -30,4 +35,4
30 35 Announcement.refreshAnnouncement();
31 36 }, 30000);
32 37 }
33 - }; No newline at end of file
38 + };
You need to be logged in to leave comments. Login now