Reverse line links in blob view (issue-309)

James Moger

2013-09-19 4f0f65fed0613cd69eba1cb8a80d0d18d0a79983

Reverse line links in blob view (issue-309)

 releases.moxie

@@ -14,6 +14,7 @@
    - Ignore the default contents of .git/description file
    changes:
    - Personal repository prefix (~) is now configurable (issue-265)
    - Reversed line links in blob view (issue-309)
    - Updated default binary and Lucene ignore extensions
    additions:
    - Added branch graph image servlet based on EGit's branch graph renderer (issue-194)

 src/main/java/com/gitblit/wicket/pages/BlobPage.java

@@ -174,7 +174,7 @@
        sb.append("<!-- start nums column -->");

        sb.append("<td id=\"nums\">");

        sb.append("<pre>");

        String numPattern = "<span id=\"L{0}\" class=\"num\">{0}</span>\n";

        String numPattern = "<span id=\"L{0}\" class=\"jump\"></span><a href=\"#L{0}\">{0}</a>\n";

        for (int i = 0; i < lines.length; i++) {

            sb.append(MessageFormat.format(numPattern, "" + (i + 1)));

        }

@@ -194,10 +194,13 @@
        

        sb.append("<table width=\"100%\"><tbody>");

        

        String linePattern = "<tr class=\"{0}\"><td><a href=\"#L{2}\">{1}</a>\r</tr>";

        String linePattern = "<tr class=\"{0}\"><td><div><span class=\"line\">{1}</span></div>\r</tr>";

        for (int i = 0; i < lines.length; i++) {

            String line = lines[i].replace('\r', ' ');

            String cssClass = (i % 2 == 0) ? "even" : "odd";

            if (StringUtils.isEmpty(line.trim())) {

                line = "&nbsp;";

            }

            sb.append(MessageFormat.format(linePattern, cssClass, line, "" + (i + 1)));

        }

        sb.append("</tbody></table></pre>");


 src/main/resources/arrow_line.png

 src/main/resources/gitblit.css

@@ -539,21 +539,29 @@
}



/* CSS trick to workaround #link topOfWindow offset problem */

#nums .num {

    border-top: 160px solid transparent;

    margin-top: -160px;

    -webkit-background-clip: padding-box;

    -moz-background-clip: padding;

    background-clip: padding-box;

#nums .jump {

    border-top: 50px solid transparent;

    margin-top: -50px;

    background: url(arrow_line.png) no-repeat scroll bottom right transparent;	
}



#nums .jump:target {

    border-left: 7px solid transparent;

    margin-right: 3px;

}



#lines .line {

    padding-left: 5px;

    color: #888;

}



#nums span:target {

#nums a:hover {

    background-color: #ffffbf;

    color: black;

    font-weight: bold;

    border-top: 1px solid red;

    border-bottom: 1px solid red;

    text-decoration: none;

}



#lines table {

@@ -568,11 +576,6 @@
    padding-left: 5px;

}



#lines a:hover {

    background-color: #ffffbf;

    text-decoration: none;

}



#lines tr:hover {

    background-color: #ffffbf;

}

@@ -583,8 +586,6 @@
#lines .even {

    background-color: #fafafa;

}







h1 small, h2 small, h3 small, h4 small, h5 small, h6 small {

    color: #888;

@@ -1152,6 +1153,8 @@
table.pretty td.graph {

    border-right: 1px solid #ddd;

    border-bottom: 1px solid #ddd;

    padding: 0px;

    /*width: 1px;*/

}



table.pretty tr.commit {