Description:
Merge branch 'master' into codejom (bug fix)
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
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 |
- |
|
|
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