From 5c0605d5665196b3726cc64bfd80b45ec963417b Mon Sep 17 00:00:00 2001
From: James Moger <james.moger@gitblit.com>
Date: Thu, 23 May 2013 23:10:43 -0400
Subject: Support relative links in commitdiff page and polish css

---
 src/main/java/com/gitblit/utils/GitBlitDiffFormatter.java  |  4 +++-
 src/main/java/com/gitblit/wicket/pages/CommitDiffPage.java | 12 ++++--------
 src/main/resources/gitblit.css                             | 14 ++++++++++++++
 3 files changed, 21 insertions(+), 9 deletions(-)

(limited to 'src')

diff --git a/src/main/java/com/gitblit/utils/GitBlitDiffFormatter.java b/src/main/java/com/gitblit/utils/GitBlitDiffFormatter.java
index 2966aa8a..f871e778 100644
--- a/src/main/java/com/gitblit/utils/GitBlitDiffFormatter.java
+++ b/src/main/java/com/gitblit/utils/GitBlitDiffFormatter.java
@@ -20,6 +20,7 @@ import static org.eclipse.jgit.lib.Constants.encode;
 import java.io.ByteArrayOutputStream;
 import java.io.IOException;
 import java.io.OutputStream;
+import java.text.MessageFormat;
 
 import org.eclipse.jgit.diff.RawText;
 import org.eclipse.jgit.util.RawParseUtils;
@@ -150,7 +151,8 @@ public class GitBlitDiffFormatter extends GitWebDiffFormatter {
 					sb.append("</tbody></table></div>\n");
 					inFile = false;
 				}
-				sb.append("<div class='header'>").append(line).append("</div>");
+				
+				sb.append(MessageFormat.format("<div class='header'><div class=\"diffHeader\" id=\"{0}\"><i class=\"icon-file\"></i> ", line)).append(line).append("</div></div>");
 				sb.append("<div class=\"diff\">");
 				sb.append("<table><tbody>");
 				inFile = true;
diff --git a/src/main/java/com/gitblit/wicket/pages/CommitDiffPage.java b/src/main/java/com/gitblit/wicket/pages/CommitDiffPage.java
index eb1d5d82..f9c4e68b 100644
--- a/src/main/java/com/gitblit/wicket/pages/CommitDiffPage.java
+++ b/src/main/java/com/gitblit/wicket/pages/CommitDiffPage.java
@@ -136,15 +136,11 @@ public class CommitDiffPage extends RepositoryPage {
 					submodulePath = submodule.gitblitPath;
 					hasSubmodule = submodule.hasSubmodule;
 
-					item.add(new LinkPanel("pathName", "list", entry.path + " @ " +
-							getShortObjectId(submoduleId), TreePage.class,
-							WicketUtils
-									.newPathParameter(submodulePath, submoduleId, "")).setEnabled(hasSubmodule));
+					// add relative link
+					item.add(new LinkPanel("pathName", "list", entry.path + " @ " + getShortObjectId(submoduleId), "#" + entry.path));
 				} else {
-					// blob
-					item.add(new LinkPanel("pathName", "list", entry.path, BlobPage.class,
-							WicketUtils
-									.newPathParameter(repositoryName, entry.commitId, entry.path)));
+					// add relative link
+					item.add(new LinkPanel("pathName", "list", entry.path, "#" + entry.path));
 				}
 
 				// quick links
diff --git a/src/main/resources/gitblit.css b/src/main/resources/gitblit.css
index 26dc4806..88f7930e 100644
--- a/src/main/resources/gitblit.css
+++ b/src/main/resources/gitblit.css
@@ -703,6 +703,16 @@ div.header {
 	border-bottom: 0;
 	border-radius: 3px 3px 0 0;
 	font-weight: bold;
+	font-family: Helvetica,arial,freesans,clean,sans-serif;
+}
+
+div.diffHeader {
+	/* CSS trick to workaround #link topOfWindow offset problem */
+    border-top: 65px solid transparent;
+    margin-top: -65px;
+    -webkit-background-clip: padding-box;
+    -moz-background-clip: padding;
+    background-clip: padding-box;
 }
 
 div.commitHeader {
@@ -858,6 +868,10 @@ div.diff.extended_header {
     font-family: inherit;
 }
 
+div.diff table {
+	border: 1px solid #ddd;
+}
+
 span.diff.add {
 	color: #008800;
 	font-family: inherit;
-- 
cgit v1.2.3