Fix styles

src/main/twirl/gitbucket/core/dashboard/issueslist.scala.html

@@ -10,14 +10,17 @@
 @import gitbucket.core.service.IssuesService
 @import gitbucket.core.service.IssuesService.IssueInfo
 <table class="table table-bordered table-hover table-issues">
+  <thead>
     <tr>
       <th style="background-color: #eee;">
         @dashboard.html.header(openCount, closedCount, condition, groups)
       </th>
     </tr>
+  </thead>
+  <tbody>
     @issues.map { case IssueInfo(issue, labels, milestone, commentCount, commitStatus) =>
       <tr>
-    <td style="padding-top: 15px; padding-bottom: 15px;">
+        <td style="padding-top: 12px; padding-bottom: 12px;">
           @if(issue.isPullRequest){
             <i class="octicon octicon-git-pull-request @(if(issue.closed) "closed" else "open")"></i>
           } else {
@@ -47,16 +50,16 @@
               </a>
             }
           </span>
-      <div class="small muted" style="margin-left: 20px; margin-top: 5px;">
+          <div class="small muted" style="margin-left: 20px; margin-top: 2px;">
             #@issue.issueId opened by @user(issue.openedUserName, styleClass="username") @datetime(issue.registeredDate)
             @milestone.map { milestone =>
-          <span style="margin: 20px;"><a href="@condition.copy(milestone = Some(Some(milestone))).toURL" class="username"><i class="octicon octicon-milestone"></i> 
+              <span style="margin: 20px;"><a href="@condition.copy(milestone = Some(Some(milestone))).toURL" class="username"><i class="octicon octicon-milestone"></i> @milestone</a></span>
-@milestone</a></span>
             }
           </div>
         </td>
       </tr>
     }
+  </tbody>
 </table>
 <div class="pull-right">
   @helper.html.paginator(page, (if(condition.state == "open") openCount else closedCount), IssuesService.IssueLimit, 10, condition.toURL)

src/main/twirl/gitbucket/core/dashboard/issuesnavi.scala.html

@@ -3,20 +3,18 @@
   condition: gitbucket.core.service.IssuesService.IssueSearchCondition)(implicit context: gitbucket.core.controller.Context)
 @import context._
 @import gitbucket.core.view.helpers._
-<ul class="nav nav-pills-group pull-left fill-width" style="margin-bottom: 10px;">
+<ul class="nav nav-pills pull-left" style="line-height: 14px;">
-  <li class="@if(filter == "created_by"){active} first">
+  <li class="@if(filter == "created_by"){active}">
     <a href="@path/dashboard/@active/created_by@condition.copy(author = None, assigned = None).toURL">Created</a>
   </li>
   <li class="@if(filter == "assigned"){active}">
     <a href="@path/dashboard/@active/assigned@condition.copy(author = None, assigned = None).toURL">Assigned</a>
   </li>
-  <li class="@if(filter == "mentioned"){active} last">
+  <li class="@if(filter == "mentioned"){active}">
     <a href="@path/dashboard/@active/mentioned@condition.copy(author = None, assigned = None).toURL">Mentioned</a>
   </li>
-  <li class="pull-right">
+</ul>
-    <form method="GET" id="search-filter-form" action="@path/dashboard/@active" style="margin-bottom: 0px;">
+<form method="GET" id="search-filter-form" action="@path/dashboard/@active" class="pull-right">
   <input type="text" id="search-filter-box" class="form-control input-lg" name="q" style="width: 400px;"
          value="is:@{if(active == "issues") "issue" else "pr"} @condition.toFilterString"/>
-    </form>
+</form>
-  </li>
-</ul>

src/main/twirl/gitbucket/core/dashboard/tab.scala.html

@@ -4,17 +4,14 @@
 <div class="dashboard-nav">
   <div class="container">
     <a href="@path/" @if(active == ""){ class="active"}>
-      <i class="octicon octicon-rss"></i>
+      <i class="octicon octicon-rss"></i> News Feed
-      News Feed
     </a>
     @if(loginAccount.isDefined){
       <a href="@path/dashboard/pulls" @if(active == "pulls" ){ class="active"}>
-        <i class="octicon octicon-git-pull-request"></i>
+        <i class="octicon octicon-git-pull-request"></i> Pull Requests
-        Pull Requests
       </a>
       <a href="@path/dashboard/issues" @if(active == "issues"){ class="active"}>
-        <i class="octicon octicon-issue-opened"></i>
+        <i class="octicon octicon-issue-opened"></i> Issues
-        Issues
       </a>
     }
   </div>
@@ -27,8 +24,6 @@ div.dashboard-nav {
   margin-bottom: 20px;
 }
 
-
-
 div.dashboard-nav a  {
   line-height: 10px;
   margin-left: 20px;
@@ -38,7 +33,6 @@ div.dashboard-nav a  {
   color: #888;
 }
 
-
 div.dashboard-nav .octicon{
     width: 16px;
     height: 16px;

src/main/twirl/gitbucket/core/issues/commentform.scala.html

@@ -24,7 +24,7 @@
         <div style="text-align: right;">
           <input type="hidden" name="issueId" value="@issue.issueId"/>
           @if((reopenable || !issue.closed) && (hasWritePermission || issue.openedUserName == loginAccount.get.userName)){
-            <input type="submit" class="btn" tabindex="3" formaction="@url(repository)/issue_comments/state" value="@{if(issue.closed) "Reopen" else "Close"}" id="action"/>
+            <input type="submit" class="btn btn-default" tabindex="3" formaction="@url(repository)/issue_comments/state" value="@{if(issue.closed) "Reopen" else "Close"}" id="action"/>
           }
           <input type="submit" class="btn btn-success" tabindex="2" formaction="@url(repository)/issue_comments/new" value="Comment"/>
         </div>

src/main/twirl/gitbucket/core/issues/labels/edit.scala.html

@@ -4,10 +4,10 @@
 @import gitbucket.core.view.helpers._
 @defining(label.map(_.labelId).getOrElse("new")){ labelId =>
 <div id="edit-label-area-@labelId">
-  <form style="margin-bottom: 0px;">
+  <form class="form-inline">
-    <input type="text" id="labelName-@labelId" style="width: 300px; margin-bottom: 0px;" value="@label.map(_.labelName)"@if(labelId == "new"){ placeholder="New label name"}/>
+    <input type="text" id="labelName-@labelId" style="width: 300px; margin-bottom: 0px;" class="form-control" value="@label.map(_.labelName)"@if(labelId == "new"){ placeholder="New label name"}/>
     <div id="label-color-@labelId" class="input-append color bscp" data-color="#@label.map(_.color).getOrElse("888888")" data-color-format="hex" style="width: 100px; margin-bottom: 0px;">
-      <input type="text" class="span3" id="labelColor-@labelId" value="#@label.map(_.color).getOrElse("888888")" readonly style="width: 100px;">
+      <input type="text" class="form-control span3" id="labelColor-@labelId" value="#@label.map(_.color).getOrElse("888888")" readonly style="width: 100px;">
       <span class="add-on"><i style="background-color: #@label.map(_.color).getOrElse("888888");"></i></span>
     </div>
     <script>
@@ -15,7 +15,7 @@
     </script>
     <span class="pull-right">
       <span id="label-error-@labelId" class="error"></span>
-      <input type="button" id="cancel-@labelId" class="btn label-edit-cancel" value="Cancel">
+      <input type="button" id="cancel-@labelId" class="btn btn-default label-edit-cancel" value="Cancel">
       <input type="button" id="submit-@labelId" class="btn btn-success" style="margin-bottom: 0px;" value="@(if(labelId == "new") "Create label"  else "Save changes")"/>
     </span>
   </form>

src/main/twirl/gitbucket/core/issues/labels/label.scala.html

@@ -6,8 +6,8 @@
 @import gitbucket.core.view.helpers._
 <tr id="label-row-@label.labelId">
   <td style="padding-top: 15px; padding-bottom: 15px;">
-    <div class="milestone row-fluid" id="label-@label.labelId">
+    <div class="milestone row" id="label-@label.labelId">
-      <div class="span8">
+      <div class="col-md-8">
         <div style="margin-top: 6px">
           <a href="@url(repository)/issues?labels=@urlEncode(label.labelName)" id="label-row-content-@label.labelId">
             <span style="background-color: #@label.color; color: #@label.fontColor; padding: 8px; font-size: 120%; border-radius: 4px;">
@@ -17,13 +17,13 @@
           </a>
         </div>
       </div>
-      <div class="@if(hasWritePermission){span2} else {span4}">
+      <div class="@if(hasWritePermission){col-md-2} else {col-md-4}">
         <div class="pull-right">
           <span class="muted">@counts.get(label.labelName).getOrElse(0) open issues</span>
         </div>
       </div>
       @if(hasWritePermission){
-        <div class="span2">
+        <div class="col-md-2">
           <div class="pull-right">
             <a href="javascript:void(0);" onclick="editLabel(@label.labelId)">Edit</a>
             &nbsp;&nbsp;

src/main/twirl/gitbucket/core/issues/labels/list.scala.html

@@ -12,11 +12,14 @@
       <tr><td></td></tr>
     </table>
     <table class="table table-bordered table-hover table-issues">
+      <thead>
         <tr id="label-row-header">
           <th style="background-color: #eee;">
             <span class="small">@labels.size labels</span>
           </th>
         </tr>
+      </thead>
+      <tbody>
         @labels.map { label =>
           @gitbucket.core.issues.labels.html.label(label, counts, repository, hasWritePermission)
         }
@@ -30,6 +33,7 @@
             </td>
           </tr>
         }
+      </tbody>
     </table>
   }
 }

src/main/twirl/gitbucket/core/issues/listparts.scala.html

@@ -173,7 +173,7 @@
     }
     @issues.map { case IssueInfo(issue, labels, milestone, commentCount, commitStatus) =>
       <tr>
-        <td style="padding-top: 15px; padding-bottom: 15px;">
+        <td style="padding-top: 12px; padding-bottom: 12px;">
           @if(hasWritePermission){
             <input type="checkbox" value="@issue.issueId"/>
           }
@@ -204,7 +204,7 @@
               </a>
             }
           </span>
-          <div class="small muted" style="margin-left: 40px; margin-top: 5px;">
+          <div class="small muted" style="margin-left: 40px; margin-top: 2px;">
             #@issue.issueId opened  @helper.html.datetimeago(issue.registeredDate) by @user(issue.openedUserName, styleClass="username")
             @milestone.map { milestone =>
               <span style="margin: 20px;"><a href="@condition.copy(milestone = Some(Some(milestone))).toURL" class="username"><i class="octicon octicon-milestone"></i> @milestone</a></span>

src/main/twirl/gitbucket/core/pulls/commits.scala.html

@@ -9,16 +9,16 @@
     <div class="muted" style="background-color: white;">
       <i class="octicon octicon-git-commit"></i> Commits on @date(day.head.commitTime)
     </div>
-    <div class="box-commits">
+    <div class="list-group box-commits">
       @day.map { commit =>
-      <div class="box-content-row" style="padding: 8px;">
+        <div class="list-group-item">
-        <ul class="nav nav-pills-group pull-right" style="margin-top: 2px; margin-bottom: 0px; margin-right: 4px;">
+          <ul class="nav nav-pills pull-right">
-          <li class="first"><a href="@url(repository)/commit/@commit.id" class="link monospace">@commit.id.substring(0, 7)</a></li>
+            <li><a href="@url(repository)/commit/@commit.id" class="link monospace" style="line-height: 16px;">@commit.id.substring(0, 7)</a></li>
-          <li class="last"><a href="@url(repository)/tree/@commit.id" style="padding-top: 9px; padding-bottom: 10px;"><i class="octicon octicon-code link"></i></a></li>
+            <li><a href="@url(repository)/tree/@commit.id" style="line-height: 16px;"><i class="octicon octicon-code link"></i></a></li>
           </ul>
           <div>
             <div class="commit-avatar-image">@avatar(commit, 40)</div>
-          <div class="commit-message-box">
+            <div>
               <a href="@url(repository)/commit/@commit.id" class="commit-message" style="font-weight: bold;">@link(commit.summary, repository)</a>
               @if(commit.description.isDefined){
                 <a href="javascript:void(0)" onclick="$('#description-@commit.id').toggle();" class="omit">...</a>
@@ -27,7 +27,7 @@
               @if(commit.description.isDefined){
                 <pre id="description-@commit.id" style="display: none;" class="commit-description">@link(commit.description.get, repository)</pre>
               }
-            <div class="small">
+              <div style="margin-top: 2px;">
                 @user(commit.authorName, commit.authorEmailAddress, "username")
                 <span class="muted">authored @helper.html.datetimeago(commit.authorTime)</span>
                 @if(commit.isDifferentFromAuthor) {

src/main/twirl/gitbucket/core/pulls/mergeguide.scala.html

@@ -48,7 +48,7 @@
         </div>
       }
       <div class="pull-right">
-        <input type="button" class="btn @if(!hasProblem){ btn-success }" id="merge-pull-request-button" value="Merge pull request"@if(hasConflict){ disabled="true"}/>
+        <input type="button" class="btn @if(!hasProblem){ btn-success }else{ btn-default }" id="merge-pull-request-button" value="Merge pull request"@if(hasConflict){ disabled="true"}/>
       </div>
       <div>
         @if(hasConflict){

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

@@ -54,15 +54,15 @@
       <a href="@url(repository)/commit/@latestCommit.id" class="commit-message">@link(latestCommit.summary, repository)</a>
       <div class="btn-group pull-right">
         @if(hasWritePermission && content.viewType == "text" && repository.branchList.contains(branch)){
-          <a class="btn btn-xs btn-default" href="@url(repository)/edit/@encodeRefName(branch)/@pathList.mkString("/")">Edit</a>
+          <a class="btn btn-sm btn-default" href="@url(repository)/edit/@encodeRefName(branch)/@pathList.mkString("/")">Edit</a>
         }
-        <a class="btn btn-xs btn-default" href="?raw=true">Raw</a>
+        <a class="btn btn-sm btn-default" href="?raw=true">Raw</a>
         @if(content.viewType == "text"){
-          <a class="btn btn-xs btn-default blame-action" href="@url(repository)/blame/@latestCommit.id/@pathList.mkString("/")" data-url="@url(repository)/get-blame/@latestCommit.id/@pathList.mkString("/")" data-repository="@url(repository)">Blame</a>
+          <a class="btn btn-sm btn-default blame-action" href="@url(repository)/blame/@latestCommit.id/@pathList.mkString("/")" data-url="@url(repository)/get-blame/@latestCommit.id/@pathList.mkString("/")" data-repository="@url(repository)">Blame</a>
         }
-        <a class="btn btn-xs btn-default" href="@url(repository)/commits/@encodeRefName(branch)/@pathList.mkString("/")">History</a>
+        <a class="btn btn-sm btn-default" href="@url(repository)/commits/@encodeRefName(branch)/@pathList.mkString("/")">History</a>
         @if(hasWritePermission && repository.branchList.contains(branch)){
-          <a class="btn btn-xs btn-danger" href="@url(repository)/remove/@encodeRefName(branch)/@pathList.mkString("/")">Delete</a>
+          <a class="btn btn-sm btn-danger" href="@url(repository)/remove/@encodeRefName(branch)/@pathList.mkString("/")">Delete</a>
         }
       </div>
     </div>

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

@@ -38,12 +38,12 @@
         <div class="muted" style="background-color: white;">
           <i class="octicon octicon-git-commit"></i> Commits on @date(day.head.commitTime)
         </div>
-        <div class="box-commits">
+        <div class="list-group box-commits">
           @day.map { commit =>
-            <div class="box-content-row" style="padding: 8px;">
+            <div class="list-group-item">
               <ul class="nav nav-pills pull-right">
-                <li class="first"><a href="@url(repository)/commit/@commit.id" class="link monospace" style="line-height: 16px;">@commit.id.substring(0, 7)</a></li>
+                <li><a href="@url(repository)/commit/@commit.id" class="link monospace" style="line-height: 16px;">@commit.id.substring(0, 7)</a></li>
-                <li class="last"><a href="@url(repository)/tree/@commit.id" style="line-height: 16px;"><i class="octicon octicon-code link"></i></a></li>
+                <li><a href="@url(repository)/tree/@commit.id" style="line-height: 16px;"><i class="octicon octicon-code link"></i></a></li>
               </ul>
               <div>
                 <div class="commit-avatar-image">@avatar(commit, 40)</div>

src/main/twirl/gitbucket/core/wiki/history.scala.html

@@ -31,14 +31,17 @@
       </li>
     </ul>
     <table class="table table-bordered fill-width pull-left">
+      <thead>
         <tr>
           <th colspan="3">
             <div class="pull-left" style="padding-top: 4px;">Revisions</div>
             <div class="pull-right">
-            <input type="button" id="compare" value="Compare Revisions" class="btn btn-mini"/>
+              <input type="button" id="compare" value="Compare Revisions" class="btn btn-dm btn-default"/>
             </div>
           </th>
         </tr>
+      </thead>
+      <tbody>
       @commits.map { commit =>
         <tr>
           <td width="0%"><input type="checkbox" name="commitId" value="@commit.id"></td>
@@ -48,6 +51,7 @@
           </td>
         </tr>
       }
+      </tbody>
     </table>
     <script>
     $(function(){

src/main/webapp/assets/common/css/gitbucket.css

@@ -372,7 +372,6 @@ ul.nav-pills i.link {
 
 div.box-commits {
   background-color: white;
-  border: 1px solid #d8d8d8;
   padding-left: 0px;
   padding-right: 0px;
   margin-top: 10px;
@@ -386,7 +385,7 @@ div.box-content {
   padding: 4px;
   border-radius: 3px;
 }
-
+/*
 div > div.box-content-row:nth-of-type(1) {
   border: none;
 }
@@ -395,7 +394,7 @@ div.box-content-row {
   border-top: 1px solid #d8d8d8;
   padding: 4px;
 }
-
+*/
 div.repo-link {
   white-space: nowrap;
   overflow: hidden;