Add a button to show forked repos to the fork account select dialog

src/main/twirl/gitbucket/core/helper/forkrepository.scala.html

@@ -1,9 +1,11 @@
 @(repository: gitbucket.core.service.RepositoryService.RepositoryInfo,
   groupAndPerm: Seq[(String, Boolean)])(implicit context: gitbucket.core.controller.Context)
 @import gitbucket.core.view.helpers
-<h2 class="facebox-header">Where should we fork this repository?</h2>
+<h2 class="facebox-header">
+  Where should we fork this repository?
+  <a class="btn btn-default btn-sm" href="@helpers.url(repository)/network/members">Show forks</a>
+</h2>
 <form action="@helpers.url(repository)/fork" id="fork" method="post">
-  <div class="content">
   <div class="block">
     @helpers.avatar(context.loginAccount.get.userName, 24) <a href="javascript:void(0);" class="js-fork-owner-select-target">@@@context.loginAccount.get.userName</a>
   </div>
@@ -18,6 +20,5 @@
       </div>
     }
   }
-  </div>
   <input id="account" name="account" type="hidden" />
 </form>

src/main/twirl/gitbucket/core/menu.scala.html

@@ -76,10 +76,19 @@
               @repositoryHeaderComponent(repository, context)
             }
             @if(repository.repository.options.allowFork) {
-              <a class="btn btn-default btn-sm" href="@helpers.url(repository)/network/members">
+              @if(context.loginAccount.isEmpty){
+                <a class="btn btn-default btn-sm" href="@context.path/signin?redirect=@helpers.urlEncode(s"${context.path}/${repository.owner}/${repository.name}")">
+                  <span class="strong"><i class="octicon octicon-repo-forked"></i>Fork</span><span class="muted">: @repository.forkedCount</span>
+                </a>
+              } else {
+                <a class="btn btn-default btn-sm" href="@context.path/@repository.owner/@repository.name/fork" rel="facebox">
                   <span class="strong"><i class="octicon octicon-repo-forked"></i>Fork</span><span class="muted">: @repository.forkedCount</span>
                 </a>
               }
+            }
+            <form id="fork-form" method="post" action="@context.path/@repository.owner/@repository.name/fork" style="display: none;">
+              <input type="hidden" name="account" value="@context.loginAccount.get.userName"/>
+            </form>
           </div>
           @gitbucket.core.helper.html.repositoryicon(repository, true)
           <a href="@helpers.url(repository.owner)">@repository.owner</a> / <a href="@helpers.url(repository)" class="strong">@repository.name</a>
@@ -97,4 +106,19 @@
     @body
   </div>
 </div>
+<script>
+$(function(){
+  $('a[rel*=facebox]').facebox({
+    'loadingImage': '@helpers.assets("/vendors/facebox/loading.gif")',
+    'closeImage': '@helpers.assets("/vendors/facebox/closelabel.png")'
+  });
+
+  $(document).on("click", ".js-fork-owner-select-target", function() {
+    var account = $(this).text().replace("@@", "");
+    $("#account").val(account);
+    $("#fork").submit();
+  });
+});
+</script>
+
 

src/main/twirl/gitbucket/core/repo/forked.scala.html

@@ -5,23 +5,7 @@
 @import gitbucket.core.view.helpers
 @gitbucket.core.html.main(s"${repository.owner}/${repository.name}", Some(repository)) {
   @gitbucket.core.html.menu("fork", repository){
-    <h1 class="body-title">
+    <h1 class="body-title">Forked repositories</h1>
-      Forked repositories
-      @if(context.loginAccount.isEmpty){
-        <a href="@context.path/signin?redirect=@helpers.urlEncode(s"${context.path}/${repository.owner}/${repository.name}")" class="btn btn-success">Fork</a>
-      } else {
-        @if(groupNames.isEmpty) {
-          <a id="fork-link" href="javascript:void(0);" class="btn btn-success">Fork</a>
-        } else {
-          <a href="@context.path/@repository.owner/@repository.name/fork" class="btn btn-success" rel="facebox">Fork</a>
-        }
-      }
-    </h1>
-    @if(context.loginAccount.isDefined && groupNames.isEmpty){
-      <form id="fork-form" method="post" action="@context.path/@repository.owner/@repository.name/fork" style="display: none;">
-        <input type="hidden" name="account" value="@context.loginAccount.get.userName"/>
-      </form>
-    }
     <div class="block">
       @if(originRepository.isDefined){
         @helpers.avatarLink(originRepository.get.owner, 20)
@@ -46,24 +30,3 @@
     }
   }
 }
-<script>
-$(function(){
-  $('a[rel*=facebox]').facebox({
-    'loadingImage': '@helpers.assets("/vendors/facebox/loading.gif")',
-    'closeImage': '@helpers.assets("/vendors/facebox/closelabel.png")'
-  });
-
-  $(document).on("click", ".js-fork-owner-select-target", function() {
-    var account = $(this).text().replace("@@", "");
-    $("#account").val(account);
-    $("#fork").submit();
-  });
-
-  @if(context.loginAccount.isDefined){
-  $(document).on("click", "a#fork-link", function(e) {
-    e.preventDefault();
-    $('#fork-form').submit();
-  });
-  }
-});
-</script>