diff --git a/src/main/scala/gitbucket/core/controller/AccountController.scala b/src/main/scala/gitbucket/core/controller/AccountController.scala index 84686e5..7663ba9 100644 --- a/src/main/scala/gitbucket/core/controller/AccountController.scala +++ b/src/main/scala/gitbucket/core/controller/AccountController.scala @@ -713,7 +713,7 @@ } helper.html.forkrepository( repository, - (groups zip managerPermissions).toMap + (groups zip managerPermissions).sortBy(_._1) ) case _ => redirect(s"/${loginUserName}") } diff --git a/src/main/twirl/gitbucket/core/helper/forkrepository.scala.html b/src/main/twirl/gitbucket/core/helper/forkrepository.scala.html index 4dfed1d..e7de31a 100644 --- a/src/main/twirl/gitbucket/core/helper/forkrepository.scala.html +++ b/src/main/twirl/gitbucket/core/helper/forkrepository.scala.html @@ -1,18 +1,23 @@ @(repository: gitbucket.core.service.RepositoryService.RepositoryInfo, - groupAndPerm: Map[String, Boolean])(implicit context: gitbucket.core.controller.Context) + groupAndPerm: Seq[(String, Boolean)])(implicit context: gitbucket.core.controller.Context) @import gitbucket.core.view.helpers

Where should we fork this repository?

-
-
@helpers.avatar(context.loginAccount.get.userName, 100)@@@context.loginAccount.get.userName
+
+
+ @helpers.avatar(context.loginAccount.get.userName, 24) @@@context.loginAccount.get.userName +
@for((groupName, isManager) <- groupAndPerm) { @if(isManager) { -
@helpers.avatar(groupName, 100)@@@groupName
+
+ @helpers.avatar(groupName, 24) @@@groupName +
} else { -
@helpers.avatar(groupName, 100)@@@groupName
+
+ @helpers.avatar(groupName, 24) @@@groupName +
} } -
-
\ No newline at end of file + diff --git a/src/main/twirl/gitbucket/core/repo/forked.scala.html b/src/main/twirl/gitbucket/core/repo/forked.scala.html index 601b031..1bd9988 100644 --- a/src/main/twirl/gitbucket/core/repo/forked.scala.html +++ b/src/main/twirl/gitbucket/core/repo/forked.scala.html @@ -54,11 +54,9 @@ }); $(document).on("click", ".js-fork-owner-select-target", function() { - if (!$(this).hasClass("disabled")) { - var account = $(this).text().replace("@@", ""); - $("#account").val(account); - $("#fork").submit(); - } + var account = $(this).text().replace("@@", ""); + $("#account").val(account); + $("#fork").submit(); }); @if(context.loginAccount.isDefined){ diff --git a/src/main/webapp/assets/vendors/facebox/facebox.css b/src/main/webapp/assets/vendors/facebox/facebox.css index 9f4aaa8..d4fac8a 100644 --- a/src/main/webapp/assets/vendors/facebox/facebox.css +++ b/src/main/webapp/assets/vendors/facebox/facebox.css @@ -80,63 +80,9 @@ } .facebox-header { - margin: -15px -15px 15px; + margin: -15px -15px 0px -15px; padding: 15px; border-bottom: 1px solid #e5e5e5; font-size: 18px; font-weight: normal; } - -.owner-select-grid { - margin-left: -8px; - overflow: hidden; -} - -.owner-select-target { - float: left; - padding: 10px; - margin: 0 8px 10px; - text-align: center; - background-color: #f2f2f2; - border-radius: 3px; -} - -.owner-select-target.enabled { - font-weight: bold; - cursor: pointer; -} - -.owner-select-target.disabled { - color: #999; - cursor: not-allowed; -} - -.owner-select-grid .avatar { - display: block; - overflow: hidden; - line-height: 1; - vertical-align: middle; - border-radius: 3px; -} - -.owner-select-target.enabled .avatar { - opacity: 1; -} - -.owner-select-target.disabled .avatar { - margin-bottom: 9px; - opacity: 0.3; -} - -.css-truncate { - display: inline-block; - max-width: 125px; - overflow: hidden; - text-overflow: ellipsis; - white-space: nowrap; - vertical-align: top; -} - -.owner-select-target .css-truncate { - max-width: 90px; -} \ No newline at end of file