[ARVADOS] updated: 3a43585742a41468abbd8a74511af6e2164bba1a

git at public.curoverse.com git at public.curoverse.com
Mon Aug 25 16:51:16 EDT 2014


Summary of changes:
 apps/workbench/app/assets/javascripts/report_issue.js             | 4 ++--
 apps/workbench/app/views/application/_report_issue_popup.html.erb | 2 +-
 apps/workbench/app/views/issue_reporter/send_report.text.erb      | 5 ++---
 3 files changed, 5 insertions(+), 6 deletions(-)

       via  3a43585742a41468abbd8a74511af6e2164bba1a (commit)
      from  e8c9386ffbd11cfcbedb63fa1681d5c0590ba7a5 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 3a43585742a41468abbd8a74511af6e2164bba1a
Author: radhika <radhika at curoverse.com>
Date:   Mon Aug 25 16:43:58 2014 -0400

    3112: add some space below the submit button and alert panel

diff --git a/apps/workbench/app/assets/javascripts/report_issue.js b/apps/workbench/app/assets/javascripts/report_issue.js
index a05cba5..15ff4bc 100644
--- a/apps/workbench/app/assets/javascripts/report_issue.js
+++ b/apps/workbench/app/assets/javascripts/report_issue.js
@@ -13,13 +13,13 @@ $(document).
         var $sendButton = $('#report-issue-submit');
         $sendButton.html('Report sent');
         $('div').remove('.modal-footer-status');
-        $('.modal-footer').append('<div class="modal-footer-status alert alert-success"><div><p align="left">Thanks for reporting this issue!</p></div></div>');
+        $('.modal-footer').append('<div><br/></div><div class="modal-footer-status alert alert-success"><p align="left">Thanks for reporting this issue!</p></div>');
       }).
       fail(function(jqxhr, status, error) {
         var $sendButton = $('#report-issue-submit');
         if ($sendButton && $sendButton.prop('disabled')) {
         $('div').remove('.modal-footer-status');
-        $('.modal-footer').append('<div class="modal-footer-status alert alert-danger"></br><p align="left">We are sorry. We could not submit your report! We really want this to work, though -- please try again.</p></div>');
+        $('.modal-footer').append('<div><br/></div><div class="modal-footer-status alert alert-danger"><p align="left">We are sorry. We could not submit your report! We really want this to work, though -- please try again.</p></div>');
         $sendButton.html('Send problem report');
         $sendButton.attr('disabled',false);
       }
diff --git a/apps/workbench/app/views/application/_report_issue_popup.html.erb b/apps/workbench/app/views/application/_report_issue_popup.html.erb
index a84fd48..fbb7a11 100644
--- a/apps/workbench/app/views/application/_report_issue_popup.html.erb
+++ b/apps/workbench/app/views/application/_report_issue_popup.html.erb
@@ -43,7 +43,7 @@
         <br/>
       </div>
 
-      <div class="modal-body" style="overflow-y:scroll">
+      <div class="modal-body" style="overflow-y:auto">
         <div class="form-group">
           <label for="support_email" class="col-sm-4 control-label"> Support email </label>
           <div class="col-sm-8">
diff --git a/apps/workbench/app/views/issue_reporter/send_report.text.erb b/apps/workbench/app/views/issue_reporter/send_report.text.erb
index 4cbb371..28841b0 100644
--- a/apps/workbench/app/views/issue_reporter/send_report.text.erb
+++ b/apps/workbench/app/views/issue_reporter/send_report.text.erb
@@ -1,13 +1,12 @@
 <% if @user %>
-Issue reported by user <%=@user.full_name%> <%=@user.email%>
+Issue reported by user <%=@user.email%>
 <% else %>
 Issue reported
 <% end %>
 
 Details of the report:
 <% if @params['report_additional_info'] %>
-<% json_map = JSON.parse(@params['report_additional_info']).map %>
-<%  map_to_s = json_map.map {|k,v| "#{k}=#{v}"}.join("\n") %>
+<%  map_to_s = JSON.parse(@params['report_additional_info']).map {|k,v| "#{k}=#{v}"}.join("\n") %>
 <%= map_to_s %>
 <% end %>
 Report text=<%=@params['report_issue_text'] %>

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list