diff options
| author | Eric Sunshine <sunshine@sunshineco.com> | 2014-12-31 10:50:08 +0100 |
|---|---|---|
| committer | Alberto Bertogli <albertito@blitiri.com.ar> | 2015-01-11 22:18:22 +0100 |
| commit | 93b161c23ea90f2dd9f7329a1afac51e23ebe3aa (patch) | |
| tree | cf989470b4af88d16a6846d8be31d92d98d59274 /views | |
| parent | 7f2f67629f1d6b7c5046ae43c1a293137c89b4d3 (diff) | |
| download | git-arr-fork-93b161c23ea90f2dd9f7329a1afac51e23ebe3aa.zip | |
views: fix broken URLs involving hierarchical branch names
Git branch names can be hierarchical (for example, "wip/parser/fix"),
however, git-arr does not take this into account when formulating URLs
on branch, tree, and blobs pages. These URLs are dysfunctional because
it is assumed incorrectly that a single "../" is sufficient to climb
over the branch name when computing relative paths to resources higher
in the hierarchy. This problem manifests as failure to load static
resources (stylesheet, etc.), broken links to commits on branch pages,
and malfunctioning breadcrumb trails.
Fix this problem by computing the the proper number of "../" based upon
the branch name, rather than assuming that a single "../" will work
unconditionally. (This is analogous to the treatment already given to
hierarchical pathnames in tree and blob views.)
Signed-off-by: Eric Sunshine <sunshine@sunshineco.com>
Signed-off-by: Alberto Bertogli <albertito@blitiri.com.ar>
Diffstat (limited to 'views')
| -rw-r--r-- | views/blob.html | 13 | ||||
| -rw-r--r-- | views/branch.html | 11 | ||||
| -rw-r--r-- | views/tree.html | 13 |
3 files changed, 21 insertions, 16 deletions
diff --git a/views/blob.html b/views/blob.html index 6e493ab..e44ff99 100644 --- a/views/blob.html +++ b/views/blob.html @@ -4,10 +4,11 @@ <head> % if not dirname.raw: -% relroot = './' +% reltree = './' % else: -% relroot = '../' * (len(dirname.split('/')) - 1) +% reltree = '../' * (len(dirname.split('/')) - 1) % end +% relroot = reltree + '../' * (len(repo.branch.split('/')) - 1) <title>git » {{repo.name}} » {{repo.branch}} » {{dirname.unicode}}/{{fname.unicode}}</title> @@ -21,13 +22,13 @@ <body class="tree"> <h1><a href="{{relroot}}../../../../../">git</a> » <a href="{{relroot}}../../../">{{repo.name}}</a> » - <a href="{{relroot}}../">{{repo.branch}}</a> » - <a href="{{relroot}}">tree</a> + <a href="{{reltree}}../">{{repo.branch}}</a> » + <a href="{{reltree}}">tree</a> </h1> <h3> - <a href="{{relroot}}">[{{repo.branch}}]</a> / -% base = smstr(relroot) + <a href="{{reltree}}">[{{repo.branch}}]</a> / +% base = smstr(reltree) % for c in dirname.split('/'): % if not c.raw: % continue diff --git a/views/branch.html b/views/branch.html index 1cc011d..0d3e891 100644 --- a/views/branch.html +++ b/views/branch.html @@ -2,14 +2,17 @@ "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd"> <html xmlns="http://www.w3.org/1999/xhtml"> <head> + +% relroot = '../' * (len(repo.branch.split('/')) - 1) + <title>git » {{repo.name}} » {{repo.branch}}</title> -<link rel="stylesheet" type="text/css" href="../../../../static/git-arr.css"/> +<link rel="stylesheet" type="text/css" href="{{relroot}}../../../../static/git-arr.css"/> <meta http-equiv="content-type" content="text/html; charset=utf-8"/> </head> <body class="branch"> -<h1><a href="../../../../">git</a> » - <a href="../../">{{repo.name}}</a> » +<h1><a href="{{relroot}}../../../../">git</a> » + <a href="{{relroot}}../../">{{repo.name}}</a> » <a href="./">{{repo.branch}}</a> </h1> @@ -35,7 +38,7 @@ % include paginate more = more, offset = offset % kwargs = dict(repo=repo, commits=commits, -% shorten=shorten, repo_root="../..") +% shorten=shorten, repo_root=relroot + "../..") % include commit-list **kwargs <p/> diff --git a/views/tree.html b/views/tree.html index 5edacb8..f983b75 100644 --- a/views/tree.html +++ b/views/tree.html @@ -4,10 +4,11 @@ <head> % if not dirname.raw: -% relroot = './' +% reltree = './' % else: -% relroot = '../' * (len(dirname.split('/')) - 1) +% reltree = '../' * (len(dirname.split('/')) - 1) % end +% relroot = reltree + '../' * (len(repo.branch.split('/')) - 1) <title>git » {{repo.name}} » {{repo.branch}} » {{dirname.unicode}}</title> @@ -19,13 +20,13 @@ <body class="tree"> <h1><a href="{{relroot}}../../../../../">git</a> » <a href="{{relroot}}../../../">{{repo.name}}</a> » - <a href="{{relroot}}../">{{repo.branch}}</a> » - <a href="{{relroot}}">tree</a> + <a href="{{reltree}}../">{{repo.branch}}</a> » + <a href="{{reltree}}">tree</a> </h1> <h3> - <a href="{{relroot}}">[{{repo.branch}}]</a> / -% base = smstr(relroot) + <a href="{{reltree}}">[{{repo.branch}}]</a> / +% base = smstr(reltree) % for c in dirname.split('/'): % if not c.raw: % continue |