From 9ced66f2080bb29c112e5ecfcc28b88f0b17987b Mon Sep 17 00:00:00 2001
From: bjarnef <bjarne_fyrstenborg@hotmail.com>
Date: Mon, 19 Jan 2015 21:49:27 +0100
Subject: [PATCH] U4-4418 - fix issue with tab sorting

Fixed issue with tab sorting, where you are able to drag the rows above
the table header. Ensured that <thead> / <th> is added, so the class
"ui-sortable" is added to <tbody> and only <tbody> will be the
drag-area.
I also added styling for delete button so it's more consistent with
other "delete"-buttons, e.g. for in settings for dropdown lists,
checkbox lists and radionbutton lists. Furthermore the delete-button is
moved inside the ItemTemplate, so the classes easily could be added to
the button and also make the button stay close to the fields.
---
 .../umbraco/controls/ContentTypeControlNew.ascx   |  6 +++---
 .../controls/ContentTypeControlNew.ascx.cs        | 15 +++++++++++++++
 2 files changed, 18 insertions(+), 3 deletions(-)

diff --git a/src/Umbraco.Web.UI/umbraco/controls/ContentTypeControlNew.ascx b/src/Umbraco.Web.UI/umbraco/controls/ContentTypeControlNew.ascx
index f59534258d..5e14be9ed9 100644
--- a/src/Umbraco.Web.UI/umbraco/controls/ContentTypeControlNew.ascx
+++ b/src/Umbraco.Web.UI/umbraco/controls/ContentTypeControlNew.ascx
@@ -25,18 +25,18 @@
     
   <cc2:Pane ID="Pane1" runat="server" Width="216" Height="80">
     <asp:DataGrid ID="dgTabs" Width="100%" runat="server" CellPadding="2" HeaderStyle-CssClass="propertyHeader"
-      ItemStyle-CssClass="propertyContent" GridLines="None" OnItemCommand="dgTabs_ItemCommand"
+      ItemStyle-CssClass="propertyContent" GridLines="None" OnItemCommand="dgTabs_ItemCommand" OnPreRender="dgTabs_PreRender"
       HeaderStyle-Font-Bold="True" AutoGenerateColumns="False" CssClass="tabs-table">
       <Columns>
         <asp:BoundColumn DataField="id" Visible="False"></asp:BoundColumn>
         <asp:TemplateColumn HeaderText="Name & sort order">
           <ItemTemplate>
-            <i class="icon-navigation handle"></i>
+            <i class="icon-navigation handle" style="display:inline-block; margin-top:5px;"></i>
             <asp:TextBox ID="txtTab" runat="server" Value='<%#DataBinder.Eval(Container.DataItem,"name")%>'></asp:TextBox>
             <asp:TextBox ID="txtSortOrder" runat="server" CssClass="sort-order" style="width:40px;background-color:#f2f2f2;" Value='<%#DataBinder.Eval(Container.DataItem,"order") %>'></asp:TextBox>
+            <asp:Button ID="btnDelete" runat="server" Text="Delete" CommandName="Delete" CssClass="btn btn-small btn-danger" />
           </ItemTemplate>
         </asp:TemplateColumn>
-        <asp:ButtonColumn ButtonType="PushButton" Text="Delete" CommandName="Delete"></asp:ButtonColumn>
       </Columns>
     </asp:DataGrid>
     <p style="text-align: center;">
diff --git a/src/Umbraco.Web.UI/umbraco/controls/ContentTypeControlNew.ascx.cs b/src/Umbraco.Web.UI/umbraco/controls/ContentTypeControlNew.ascx.cs
index 096bb5b724..39f04b0688 100644
--- a/src/Umbraco.Web.UI/umbraco/controls/ContentTypeControlNew.ascx.cs
+++ b/src/Umbraco.Web.UI/umbraco/controls/ContentTypeControlNew.ascx.cs
@@ -28,6 +28,21 @@ namespace Umbraco.Web.UI.Umbraco.Controls
             DataTypeControllerUrl = Url.GetUmbracoApiServiceBaseUrl<DataTypeController>(x => x.GetById(0));
             ContentTypeControllerUrl = Url.GetUmbracoApiServiceBaseUrl<ContentTypeController>(x => x.GetAssignedListViewDataType(0));
         }
+
+        protected void dgTabs_PreRender(object sender, EventArgs e)
+        {
+            dgTabs.UseAccessibleHeader = true; //to make sure we render th, not td
+
+            Table table = dgTabs.Controls[0] as Table;
+            if (table != null && table.Rows.Count > 0)
+            {
+                // here we render <thead> and <tfoot>
+                if (dgTabs.ShowHeader) 
+                    table.Rows[0].TableSection = TableRowSection.TableHeader;
+                if (dgTabs.ShowFooter)
+                    table.Rows[table.Rows.Count - 1].TableSection = TableRowSection.TableFooter;
+            }
+        }
       
     }
 }
\ No newline at end of file