SONAR-3009 Widgets can not be configured on IE9

4 files changed, 22 insertions, 17 deletions

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/_configure_widget.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/_configure_widget.html.erb
index c27485fcf58..78196d47796 100644
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/_configure_widget.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/_configure_widget.html.erb
@@ -8,12 +8,16 @@
    end
 %>
 
-<div class="handle" style="//overflow:hidden;//zoom:1;">
-  <a class="block-remove" onclick="portal.deleteWidget(this);return false;"><%= message('delete') -%></a>
-  <% if widget.java_definition.isEditable() %>
-    <a class="block-view-toggle" onclick="portal.editWidget(<%= widget.id -%>);return false;"><%= message('edit') -%></a>
-  <% end %>
-  <%= h message('widget.' + widget.java_definition.getId() + '.name', :default => widget.java_definition.getTitle()) -%>
+<div class="widget-header">
+  <div class="widget-actions">
+    <% if widget.java_definition.isEditable() %>
+      <a class="link-action" onclick="portal.editWidget(<%= widget.id -%>);return false;"><%= message('edit') -%></a>
+    <% end %>
+    <a class="link-action" onclick="portal.deleteWidget(this);return false;"><%= message('delete') -%></a>
+  </div>
+  <div class="widget-handle">
+    <%= h message('widget.' + widget.java_definition.getId() + '.name', :default => widget.java_definition.getTitle()) -%>
+  </div>
 </div>
 
 
@@ -22,7 +26,7 @@
 </div>
 
 
-<div id="widget_<%= widget.id -%>" class="configure_widget <%= widget.java_definition.getId() -%>" style="height:100%;<%= 'display:none;' if !widget.configured -%>">
+<div id="widget_<%= widget.id -%>" class="configure_widget widget-<%= widget.java_definition.getId() -%>" style="height:100%;<%= 'display:none;' if !widget.configured -%>">
   <!--[if lte IE 6]>
   <style type="text/css">
     #dashboard .block .content .transparent {
@@ -40,7 +44,8 @@
     <% end %>
     <%= widget_body -%>
     <% if default_layout %>
-      <div class="clear"> </div></div>
+      <div class="clear"></div>
+      </div>
     <% end %>
   <% else %>
     <div class="widget"><p><%= message('no_data') -%></p></div>
diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/_widget_properties.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/_widget_properties.html.erb
index e071c6ce7e1..4432a095d6f 100644
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/_widget_properties.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/_widget_properties.html.erb
@@ -3,16 +3,16 @@
                    :update => {:failure => "error#{widget.id}"},
                    :failure => "$('error#{widget.id}').show()" do -%>
   <div id="error<%= widget.id -%>" class="error" style="display: none"></div>
-  <table class="form width100">
+  <table class="table width100">
     <tbody>
     <% widget.java_definition.getWidgetProperties().each do |property_def| %>
       <tr>
-        <td valign="top" class="thin nowrap"><b><%= property_def.key() -%></b><%= "*" unless property_def.optional()==true -%>:</td>
-        <td id="row_<%= property_def.key() -%>">
+        <td class="form-key-cell"><%= property_def.key() -%><%= "*" unless property_def.optional()==true -%></td>
+        <td class="form-val-cell" id="row_<%= property_def.key() -%>">
           <%= property_value_field(property_def, widget.property_text_value(property_def.key())) -%>
-          <span class="note">
-                <%= message("widget." + widget.key + ".param." + property_def.key(), :default => property_def.description()) -%>
-              </span>
+          <div class="form-val-note">
+            <%= message("widget." + widget.key + ".param." + property_def.key(), :default => property_def.description()) -%>
+          </div>
         </td>
       </tr>
     <% end %>
diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/configure.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/configure.html.erb
index 12a599e1f3d..8e38d90656f 100644
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/configure.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/configure.html.erb
@@ -6,8 +6,8 @@
     column:'dashboard-column',
     columnhandle:'column-handle',
     block:'block',
-    handle:'none',
     hoverclass:'block-hover',
+    handleClass: 'widget-handle',
     dashboardstate:'dashboardstate',
     toggle:'block-toggle',
     blocklist:'widget_defs',
@@ -49,7 +49,7 @@
      columns=@dashboard.column_layout.split('-')
      for index in 1..columns.size()
   %>
-    <div class="dashboard-column-wrapper" style="width: <%= columns[index-1] -%>">
+    <div class="dashboard-column-wrapper" style="width: <%= columns[index-1] -%>;margin: 0 -1px 0 0;">
       <div class="dashboard-column" id="dashboard-column-<%= index -%>" style="margin: 0 <%= index<columns.size() ? "5px" : "0px" -%> 0 <%= index>1 ? "5px" : "0px" -%>;">
         <div class="column-handle" style="display: none"></div>
 
diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/treemap/_treemap_container.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/treemap/_treemap_container.html.erb
index e8381ed712a..2afabddf5e7 100644
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/treemap/_treemap_container.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/treemap/_treemap_container.html.erb
@@ -24,7 +24,7 @@
 <div id="tm-<%= treemap_id -%>" class="spacer-bottom"></div>
 
 <div style="margin: 5px 0 0 0" class="notes">
-  <div style="float: right;cursor: help"><%= image_tag 'help_gray.gif', :title => h(message('treemap.click_help')) -%></div>
+  <div style="float: right;cursor: help"><%= image_tag 'help.png', :title => h(message('treemap.click_help')) -%></div>
   <div id="tm-bc-<%= treemap_id -%>">/</div>
 </div>