From 8600e789a634cb0117adc911b56eac7f2aba9522 Mon Sep 17 00:00:00 2001
From: Fabio Teixeira <fabio1079@gmail.com>
Date: Fri, 18 Dec 2015 15:54:14 -0200
Subject: [PATCH] Fix rating form arrow bug

---
 src/noosfero-spb/gov_user/public/views/create-institution.js |  6 ++----
 src/noosfero-spb/noosfero-spb-theme/css/use-report.css       |  7 ++++++-
 src/noosfero-spb/noosfero-spb-theme/theme.js                 | 44 +++++++++++++++++++++++---------------------
 3 files changed, 31 insertions(+), 26 deletions(-)

diff --git a/src/noosfero-spb/gov_user/public/views/create-institution.js b/src/noosfero-spb/gov_user/public/views/create-institution.js
index 7db3634..8082693 100644
--- a/src/noosfero-spb/gov_user/public/views/create-institution.js
+++ b/src/noosfero-spb/gov_user/public/views/create-institution.js
@@ -200,11 +200,9 @@ modulejs.define('CreateInstitution', ['jquery', 'NoosferoRoot', 'SelectElement']
 
   function toggle_extra_fields_style_status(status) {
       if(status) {
-        $('.extra-fields-container').css({ marginTop: "55px" });
-        $('.button-bar').css({ marginTop: "55px" });
+        $('.comments-software-extra-fields').css({height: "180px" });
       } else {
-        $('.extra-fields-container').css({ marginTop: "0px" });
-        $('.button-bar').css({ marginTop: "20px" });
+        $('.comments-software-extra-fields').css({height: "140px" });
       }
   }
 
diff --git a/src/noosfero-spb/noosfero-spb-theme/css/use-report.css b/src/noosfero-spb/noosfero-spb-theme/css/use-report.css
index dfa6755..0e68b70 100644
--- a/src/noosfero-spb/noosfero-spb-theme/css/use-report.css
+++ b/src/noosfero-spb/noosfero-spb-theme/css/use-report.css
@@ -237,6 +237,11 @@
   margin-left: 10px;
 }
 
+#content .star-rate-form .star-comment-container .extra-fields-container{
+  height: 56px;
+  overflow: hidden;
+}
+
 #content .star-rate-form .star-comment-container .comments-software-extra-fields {
   height: 0;
   overflow: initial;
@@ -245,7 +250,7 @@
 #content .star-rate-form .star-comment-container .comments-software-extra-fields #input_institution_comments {
   margin-bottom: 19px;
   margin-top: 16px;
-  height: 50px;
+  height: auto;
 }
 
 #content .star-rate-form .star-comment-container .comments-software-extra-fields #input_institution_comments > label {
diff --git a/src/noosfero-spb/noosfero-spb-theme/theme.js b/src/noosfero-spb/noosfero-spb-theme/theme.js
index fb61d81..83ec5ac 100644
--- a/src/noosfero-spb/noosfero-spb-theme/theme.js
+++ b/src/noosfero-spb/noosfero-spb-theme/theme.js
@@ -51,7 +51,7 @@ function alignBlocks(containerIndex){
 // Foco no botao de busca
 
 $('#link-buscar').click(function(e) {
-    e.preventDefault();
+    e.defaultPrevented();
     window.location.hash = '#portal-searchbox';
     $('.searchField').focus()
 })
@@ -195,35 +195,37 @@ $('#link-buscar').click(function(e) {
     $('.star-tooltip').html("?");
   }
 
-  // TODO: fix calls for this function below
-  // TODO: comments-additional-information --> comments-display-fields
+
   function set_arrow_direction() {
     var additional_data_bar = $('.comments-display-fields');
-    var arrow = $('.comments-arrow-down');
-    var state = 0;
+
     additional_data_bar.on('click', function() {
-      animateExtraFields();
+      var arrow = additional_data_bar.find('span[class*="comments-arrow"]');
+      var additional_fields = $('.comments-software-extra-fields');
+
+      if (additional_fields) {
+        animateExtraFields(additional_fields, arrow);
+      }
     });
   }
 
-  function animateExtraFields() {
-    var additional_data_bar = $('.comments-display-fields');
-    var arrow = ($('.comments-arrow-down')[0])? $('.comments-arrow-down') : $('.comments-arrow-up');
-    console.log(arrow);
-    var fields = $('.comments-software-extra-fields');
-    if(fields) {
-      var innerHeight = fields[0].offsetHeight;
-      if(fields.height()!==0) {
-        arrow.attr('class', "comments-arrow-down");
-        fields.animate({height: 0});
-      }
-      else {
-        arrow.attr('class', "comments-arrow-up");
-        fields.animate({height: 140});
-      }
+
+  function animateExtraFields(additional_fields, arrow) {
+    var innerHeight = additional_fields[0].offsetHeight;
+
+    if(additional_fields.height() !== 0) {
+      arrow.attr('class', "comments-arrow-down");
+      additional_fields.animate({height: 0});
+    } else {
+      arrow.attr('class', "comments-arrow-up");
+      additional_fields.animate({height: additional_fields.get(0).scrollHeight}, 1000 );
     }
+
+    // Fix for the arrow change on modal display to block, killing the entire page
+    $("#institution_modal").css({'display':'none'});
   }
 
+
   function set_use_report_content() {
     $('.profile-homepage .organization-average-rating-container .rate-this-organization a').html('Avalie este software');
     $('.make-report-block .make-report-container .button-bar a span').html('Avalie este software');
--
libgit2 0.21.2