one diff less

Author: 6543

templates/repo/settings/branches.tmpl

@@ -39,23 +39,21 @@
 			<div class="ui attached segment">
 				<div class="flex-list" id="protected-branches-list" data-update-priority-url="{{$.Repository.Link}}/settings/branches/priority">
 					{{range .ProtectedBranches}}
-						<div data-id="{{.ID}}">
-							<div class="flex-item tw-items-center">
-								<div class="drag-handle tw-cursor-grab">
-									{{svg "octicon-grabber" 16}}
-								</div>
-								<div class="flex-item-main">
-									<div class="flex-item-title">
-										<div class="ui basic primary label">{{.RuleName}}</div>
-									</div>
-								</div>
-								<div class="flex-item-trailing">
-									<a class="rm ui tiny button" href="{{$.Repository.Link}}/settings/branches/edit?rule_name={{.RuleName}}">{{ctx.Locale.Tr "repo.settings.edit_protected_branch"}}</a>
-									<button class="ui red tiny button delete-button" data-url="{{$.Repository.Link}}/settings/branches/{{.ID}}/delete" data-id="{{.ID}}">
-										{{ctx.Locale.Tr "repo.settings.protected_branch.delete_rule"}}
-									</button>
+					  <div class="flex-item tw-items-center item" data-id="{{.ID}}" >
+							<div class="drag-handle tw-cursor-grab">
+								{{svg "octicon-grabber" 16}}
+							</div>
+							<div class="flex-item-main">
+								<div class="flex-item-title">
+									<div class="ui basic primary label">{{.RuleName}}</div>
 								</div>
 							</div>
+							<div class="flex-item-trailing">
+								<a class="rm ui tiny button" href="{{$.Repository.Link}}/settings/branches/edit?rule_name={{.RuleName}}">{{ctx.Locale.Tr "repo.settings.edit_protected_branch"}}</a>
+								<button class="ui red tiny button delete-button" data-url="{{$.Repository.Link}}/settings/branches/{{.ID}}/delete" data-id="{{.ID}}">
+									{{ctx.Locale.Tr "repo.settings.protected_branch.delete_rule"}}
+								</button>
+							</div>
 						</div>
 					{{else}}
 						<div class="flex-item center aligned">

web_src/js/features/repo-settings-branches.test.ts

@@ -15,20 +15,14 @@ describe('Repository Branch Settings', () => {
   beforeEach(() => {
     document.body.innerHTML = `
       <div id="protected-branches-list" data-update-priority-url="some/repo/branches/priority">
-        <div data-id="1">
-          <div class="flex-item">
-            <div class="drag-handle"></div>
-          </div>
+        <div class="flex-item tw-items-center item" data-id="1" >
+          <div class="drag-handle"></div>
         </div>
-        <div data-id="2">
-          <div class="flex-item">
-            <div class="drag-handle"></div>
-          </div>
+        <div class="flex-item tw-items-center item" data-id="2" >
+          <div class="drag-handle"></div>
         </div>
-        <div data-id="3">
-          <div class="flex-item">
-            <div class="drag-handle"></div>
-          </div>
+        <div class="flex-item tw-items-center item" data-id="3" >
+          <div class="drag-handle"></div>
         </div>
       </div>
     `;