Have report email alerts appear somewhere sensible in mobile view.

mysociety · Mar 10, 2012 · ac38521 · ac38521
1 parent ce86284
commit ac38521
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 8 deletions.
diff --git a/templates/web/fixmystreet/report/display.html b/templates/web/fixmystreet/report/display.html
@@ -22,6 +22,13 @@
 
 [% INCLUDE 'report/_main.html' %]
 
+<div class="shadow-wrap">
+    <ul id="key-tools">
+        <li><a rel="nofollow" id="key-tool-report-abuse" class="abuse" href="[% c.uri_for( '/contact', { id => problem.id } ) %]">[% loc('Report abuse' ) %]</a></li>
+        <li><a rel="nofollow" id="key-tool-report-updates" class="feed" href="[% c.uri_for( '/alert/subscribe', { id => problem.id } ) %]">[% loc('Get updates' ) %]</a></li>
+        <li><a class="chevron" id="key-tool-problems-nearby" href="[% c.uri_for( '/around', { lat => short_latitude, lon => short_longitude } ) %]">[% loc( 'Problems nearby' ) %]</a></li>
+    </ul>
+
 <div id="report-updates-data" class="hidden-js">
     <form action="[% c.uri_for( '/alert/subscribe' ) %]" method="post">
         <p><a href="[% c.uri_for( '/rss', problem.id ) %]">
@@ -40,13 +47,6 @@
     </form>
 </div>
 
-<div class="shadow-wrap">
-    <ul id="key-tools">
-        <li><a rel="nofollow" id="key-tool-report-abuse" class="abuse" href="[% c.uri_for( '/contact', { id => problem.id } ) %]">[% loc('Report abuse' ) %]</a></li>
-        <li><a rel="nofollow" id="key-tool-report-updates" class="feed" href="[% c.uri_for( '/alert/subscribe', { id => problem.id } ) %]">[% loc('Get updates' ) %]</a></li>
-        <li><a class="chevron" id="key-tool-problems-nearby" href="[% c.uri_for( '/around', { lat => short_latitude, lon => short_longitude } ) %]">[% loc( 'Problems nearby' ) %]</a></li>
-    </ul>
-
 </div>
 
 [% INCLUDE 'report/updates.html' %]

diff --git a/web/cobrands/fixmystreet/fixmystreet.js b/web/cobrands/fixmystreet/fixmystreet.js
@@ -377,7 +377,7 @@ $.fn.small_drawer = function(id) {
             d.hide().removeClass('hidden-js').css({
                 padding: '1em',
                 background: '#fff'
-            }).appendTo($('.shadow-wrap'));
+            });
             $this.data('setup', true);
         }
         d.slideDown();
@@ -464,6 +464,7 @@ $.fn.drawer = function(id, ajax) {
     if($('.mobile').length){
         $('#map_permalink').hide();
         $('#key-tools a.feed').appendTo('#sub_map_links');
+        $('#report-updates-data').insertAfter($('#map_box'));
     }
     //add open/close toggle button on desk
     $('#sub_map_links').prepend('<span id="map_links_toggle">&nbsp;</span>');