supports download missing image button on first image while coding

montera34 · May 14, 2013 · 36f610c · 36f610c
1 parent fdced18
commit 36f610c
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 7 deletions.
diff --git a/app/assets/javascripts/coding.js b/app/assets/javascripts/coding.js
@@ -34,17 +34,16 @@ $(document).ready(function () {
         renderHighlightedAreas();
 
         // after we slide for the next image, we normally initialize the imgAreaSelect for the new image, but before that, we also check if the user have a premonition
-        $('#downloadImageAlert').hide();
         if (pageData.allowedToCode) {
             // is the image was not found, it will not initialize the imgAreaSelect, and this not working in the heroku deployed version
             if ( (current_img.attr("alt") == "404") || (current_img.attr('data-missing')=='true') ) {
                 $('#images_section div.active img').imgAreaSelect({handles: true,onSelectEnd: highlightingArea, disable:true});
                 $('#missing_image_id').val(current_img.attr('data-id'));
-                $('#downloadImageAlert').show();
             }else{
                 $('#images_section div.active img').imgAreaSelect({handles: true,onSelectEnd: highlightingArea});
             }
         }
+        setMissingImageInfoFromImg(current_img);
 
         // update the sidebar meta-data about the image
         var source_url = $("#images_section div.active img").attr('url');
@@ -99,6 +98,15 @@ $(document).ready(function () {
 
 });
 
+function setMissingImageInfoFromImg(imgElem){
+    if(imgElem.attr('data-missing')=='true'){
+        $('#missing_image_id').val(imgElem.attr('data-id'));
+        $('#downloadImageAlert').show();
+    } else {
+        $('#downloadImageAlert').hide();
+    }
+}
+
 function markAsNothingToCode(){ 
     var current_img = getCurrentImage();
     // Clear existing areas

diff --git a/app/views/coding/process_images.html.erb b/app/views/coding/process_images.html.erb
@@ -227,19 +227,21 @@ $(function() {
     $("#coding").css("color","#fff")
     $("#coding").attr("class", "btn btn-info");
 
+    // hide alerts
+    $('#youHaveLoopedAlert').hide();
+    $('#downloadImageAlert').hide();
+
     // support auto-starting on the image the user clicked in the display view
     if (pageData.startImageName.length != 0) {
         $("#images_section div.active img").parent().attr("class", "item")
-        $("#images_section div img[name='"+pageData.startImageName+"']").parent().attr("class", "active item")
+        var startImage = $("#images_section div img[name='"+pageData.startImageName+"']");
+        startImage.parent().attr("class", "active item");
+        setMissingImageInfoFromImg(startImage);
     };
 
     // when the user click save, it will set the status to "0", to allow the form to be submitted
     $(".save").click(function (e) { setModified(0) })
 
-    // hide alerts
-    $('#youHaveLoopedAlert').hide();
-    $('#downloadImageAlert').hide();
-
     $(document).bind('keydown', 'left', function(){ $('#carouselPrevLink').click() });
     $(document).bind('keydown', 'right', function(){ $('#carouselNextLink').click() });
     $(document).bind('keydown', 'n', markAsNothingToCode);