diff --git a/src/main/scala/gitbucket/core/controller/WikiController.scala b/src/main/scala/gitbucket/core/controller/WikiController.scala index 7f876d0..625ba34 100644 --- a/src/main/scala/gitbucket/core/controller/WikiController.scala +++ b/src/main/scala/gitbucket/core/controller/WikiController.scala @@ -38,7 +38,8 @@ get("/:owner/:repository/wiki")(referrersOnly { repository => getWikiPage(repository.owner, repository.name, "Home").map { page => html.page("Home", page, getWikiPageList(repository.owner, repository.name), - repository, hasWritePermission(repository.owner, repository.name, context.loginAccount)) + repository, hasWritePermission(repository.owner, repository.name, context.loginAccount), + getWikiSideBar(repository.owner, repository.name)) } getOrElse redirect(s"/${repository.owner}/${repository.name}/wiki/Home/_edit") }) @@ -47,7 +48,8 @@ getWikiPage(repository.owner, repository.name, pageName).map { page => html.page(pageName, page, getWikiPageList(repository.owner, repository.name), - repository, hasWritePermission(repository.owner, repository.name, context.loginAccount)) + repository, hasWritePermission(repository.owner, repository.name, context.loginAccount), + getWikiSideBar(repository.owner, repository.name)) } getOrElse redirect(s"/${repository.owner}/${repository.name}/wiki/${StringUtil.urlEncode(pageName)}/_edit") }) @@ -124,7 +126,11 @@ updateLastActivityDate(repository.owner, repository.name) recordEditWikiPageActivity(repository.owner, repository.name, loginAccount.userName, form.pageName, commitId) } - redirect(s"/${repository.owner}/${repository.name}/wiki/${StringUtil.urlEncode(form.pageName)}") + if(notReservedPageName(form.pageName)) { + redirect(s"/${repository.owner}/${repository.name}/wiki/${StringUtil.urlEncode(form.pageName)}") + } else { + redirect(s"/${repository.owner}/${repository.name}/wiki") + } } }) @@ -140,7 +146,11 @@ updateLastActivityDate(repository.owner, repository.name) recordCreateWikiPageActivity(repository.owner, repository.name, loginAccount.userName, form.pageName) - redirect(s"/${repository.owner}/${repository.name}/wiki/${StringUtil.urlEncode(form.pageName)}") + if(notReservedPageName(form.pageName)) { + redirect(s"/${repository.owner}/${repository.name}/wiki/${StringUtil.urlEncode(form.pageName)}") + } else { + redirect(s"/${repository.owner}/${repository.name}/wiki") + } } }) @@ -186,13 +196,15 @@ override def validate(name: String, value: String, messages: Messages): Option[String] = if(value.exists("\\/:*?\"<>|".contains(_))){ Some(s"${name} contains invalid character.") - } else if(value.startsWith("_") || value.startsWith("-")){ + } else if(notReservedPageName(value) && (value.startsWith("_") || value.startsWith("-"))){ Some(s"${name} starts with invalid character.") } else { None } } + private def notReservedPageName(value: String) = value != "_Sidebar" + private def conflictForNew: Constraint = new Constraint(){ override def validate(name: String, value: String, messages: Messages): Option[String] = { targetWikiPage.map { _ => diff --git a/src/main/twirl/gitbucket/core/wiki/page.scala.html b/src/main/twirl/gitbucket/core/wiki/page.scala.html index 99afb54..e9700d9 100644 --- a/src/main/twirl/gitbucket/core/wiki/page.scala.html +++ b/src/main/twirl/gitbucket/core/wiki/page.scala.html @@ -2,7 +2,8 @@ page: gitbucket.core.service.WikiService.WikiPageInfo, pages: List[String], repository: gitbucket.core.service.RepositoryService.RepositoryInfo, - hasWritePermission: Boolean)(implicit context: gitbucket.core.controller.Context) + hasWritePermission: Boolean, + sidebar: Option[gitbucket.core.service.WikiService.WikiPageInfo])(implicit context: gitbucket.core.controller.Context) @import context._ @import gitbucket.core.view.helpers._ @import gitbucket.core.service.WikiService._ @@ -26,10 +27,14 @@
@defining(15){ max => -
- Pages @pages.length -
-
+ +
+ + + Pages @pages.length +
+
+
@pages.zipWithIndex.map { case (page, i) => } + @sidebar.map { sidebarPage => +
+ @if(hasWritePermission){ + + + + } + @markdown(sidebarPage.content, repository, true, false, false, false, pages) +
+ }.getOrElse{ + @if(hasWritePermission){ + +
+ + Add a custom sidebar +
+
+ } + }
Clone this wiki locally
@@ -68,6 +92,24 @@ $(e.target).parents('div.show-more').remove(); }); + $('#show-pages-index').click(function(e){ + if($('#pages-index').is(":visible")){ + $('#triangle-down').hide(); + $('#triangle-right').show(); + $('#pages-index').hide(); + } else { + $('#triangle-right').hide(); + $('#triangle-down').show(); + $('#pages-index').show(); + } + }); + + @sidebar.map { sidebarPage => + $('#pages-index').hide(); + $('#triangle-down').hide(); + $('#triangle-right').show(); + } + @if(settings.ssh && loginAccount.isDefined){ $('#repository-url-http').click(function(){ $('#repository-url').val('@httpUrl(repository)'); diff --git a/src/main/webapp/assets/common/css/gitbucket.css b/src/main/webapp/assets/common/css/gitbucket.css index cc9f069..f1bb307 100644 --- a/src/main/webapp/assets/common/css/gitbucket.css +++ b/src/main/webapp/assets/common/css/gitbucket.css @@ -1175,6 +1175,36 @@ color: #c00; } +div.wiki-index-header { + background-color: #f5f5f5; + color: #333333; + margin: 0; + border-top-left-radius: 3px; + border-top-right-radius: 3px; + border: 1px solid #d8d8d8; + padding: 8px 8px 8px 8px; +} + +div.wiki-sidebar { + background-color: white; + border: 1px solid #d8d8d8; + padding: 4px; + border-bottom-left-radius: 3px; + border-bottom-right-radius: 3px; + margin-bottom: 20px; + margin-top: 20px; +} + +div.wiki-index-content { + background-color: white; + border: 1px solid #d8d8d8; + padding: 0px; + border-bottom-left-radius: 3px; + border-bottom-right-radius: 3px; + margin-bottom: 20px; + border-top: none; +} + /****************************************************************************/ /* Commit */ /****************************************************************************/