From c8780984f8c211342acf6b133bd08b940cb9cba2 Mon Sep 17 00:00:00 2001 From: Cirno the Strongest <1447794+CirnoT@users.noreply.github.com> Date: Mon, 15 Jun 2020 20:39:39 +0200 Subject: [PATCH] Rework blame template and styling (#11885) * Rework blame template and styling * cleanup template and add missing NumLines Co-authored-by: techknowlogick --- routers/repo/blame.go | 30 ++++---------- templates/repo/blame.tmpl | 74 ++++++++++++++--------------------- templates/repo/home.tmpl | 2 +- web_src/less/_base.less | 13 ++++-- web_src/less/_repository.less | 8 +++- 5 files changed, 54 insertions(+), 73 deletions(-) diff --git a/routers/repo/blame.go b/routers/repo/blame.go index 00ef9a99ea..d353f3b5df 100644 --- a/routers/repo/blame.go +++ b/routers/repo/blame.go @@ -118,29 +118,15 @@ func RefBlame(ctx *context.Context) { ctx.Data["IsBlame"] = true - if ctx.Repo.CanEnableEditor() { - // Check LFS Lock - lfsLock, err := ctx.Repo.Repository.GetTreePathLock(ctx.Repo.TreePath) - if err != nil { - ctx.ServerError("GetTreePathLock", err) - return - } - if lfsLock != nil && lfsLock.OwnerID != ctx.User.ID { - ctx.Data["CanDeleteFile"] = false - ctx.Data["DeleteFileTooltip"] = ctx.Tr("repo.editor.this_file_locked") - } else { - ctx.Data["CanDeleteFile"] = true - ctx.Data["DeleteFileTooltip"] = ctx.Tr("repo.editor.delete_this_file") - } - } else if !ctx.Repo.IsViewBranch { - ctx.Data["DeleteFileTooltip"] = ctx.Tr("repo.editor.must_be_on_a_branch") - } else if !ctx.Repo.CanWrite(models.UnitTypeCode) { - ctx.Data["DeleteFileTooltip"] = ctx.Tr("repo.editor.must_have_write_access") - } - ctx.Data["FileSize"] = blob.Size() ctx.Data["FileName"] = blob.Name() + ctx.Data["NumLines"], err = blob.GetBlobLineCount() + if err != nil { + ctx.NotFound("GetBlobLineCount", err) + return + } + blameReader, err := git.CreateBlameReader(models.RepoPath(userName, repoName), commitID, fileName) if err != nil { ctx.NotFound("CreateBlameReader", err) @@ -245,9 +231,9 @@ func renderBlame(ctx *context.Context, blameParts []git.BlamePart, commitNames m //Line number if len(part.Lines)-1 == index && len(blameParts)-1 != pi { - lineNumbers.WriteString(fmt.Sprintf(`%d`, i, i)) + lineNumbers.WriteString(fmt.Sprintf(``, i, i)) } else { - lineNumbers.WriteString(fmt.Sprintf(`%d`, i, i)) + lineNumbers.WriteString(fmt.Sprintf(``, i, i)) } //Code line diff --git a/templates/repo/blame.tmpl b/templates/repo/blame.tmpl index c2410e8c1d..84e3e8c53e 100644 --- a/templates/repo/blame.tmpl +++ b/templates/repo/blame.tmpl @@ -1,51 +1,37 @@ -
- -

-
-
- - {{.FileName}} {{FileSize .FileSize}}{{if .IsLFSFile}} ({{.i18n.Tr "repo.stored_lfs"}}){{end}} +
+

+
+
+
+ {{.NumLines}} {{.i18n.Tr (TrN .i18n.Lang .NumLines "repo.line" "repo.lines") }} +
+
{{FileSize .FileSize}}
-
-
-
- {{.i18n.Tr "repo.file_raw"}} - {{if not .IsViewCommit}} - {{.i18n.Tr "repo.file_permalink"}} - {{end}} - {{.i18n.Tr "repo.normal_view"}} - {{.i18n.Tr "repo.file_history"}} -
- {{if .Repository.CanEnableEditor}} - {{if .CanEditFile}} - {{svg "octicon-pencil" 16}} - {{else}} - {{svg "octicon-pencil" 16}} - {{end}} - {{if .CanDeleteFile}} - {{svg "octicon-trashcan" 16}} - {{else}} - {{svg "octicon-trashcan" 16}} - {{end}} - {{end}} -
+
+
+
+
+ {{.i18n.Tr "repo.file_raw"}} + {{if not .IsViewCommit}} + {{.i18n.Tr "repo.file_permalink"}} + {{end}} + {{.i18n.Tr "repo.normal_view"}} + {{.i18n.Tr "repo.file_history"}} +

-
-
- - - - - - - - -
{{.BlameCommitInfo}}{{.BlameLineNums}}
    {{.BlameContent}}
-
+
+ + + + + + + + +
{{.BlameCommitInfo}}{{.BlameLineNums}}
    {{.BlameContent}}
+
- -
diff --git a/templates/repo/home.tmpl b/templates/repo/home.tmpl index 3a0d364f34..45f51819f8 100644 --- a/templates/repo/home.tmpl +++ b/templates/repo/home.tmpl @@ -1,5 +1,5 @@ {{template "base/head" .}} -
+
{{template "repo/header" .}}
{{template "base/alert" .}} diff --git a/web_src/less/_base.less b/web_src/less/_base.less index b88b4a8a55..a80806ec20 100644 --- a/web_src/less/_base.less +++ b/web_src/less/_base.less @@ -1069,7 +1069,6 @@ i.icons { } .lines-num { - vertical-align: top; text-align: right !important; color: #999999; background: #f5f5f5; @@ -1077,7 +1076,13 @@ i.icons { user-select: none; span { - &:before { + &.bottom-line { + &:after { + border-bottom: 1px solid #eaecef; + } + } + + &:after { content: attr(data-line-number); line-height: 20px !important; padding: 0 10px; @@ -1090,6 +1095,7 @@ i.icons { .lines-num, .lines-code { padding: 0 !important; + vertical-align: top; pre, ol, @@ -1101,7 +1107,7 @@ i.icons { li { display: block; width: calc(100% - 1ch); - margin-left: 1ch; + padding-left: 1ch; } } } @@ -1149,7 +1155,6 @@ i.icons { } } -.lines-num, .lines-code, .lines-commit { .bottom-line { diff --git a/web_src/less/_repository.less b/web_src/less/_repository.less index f37eea3ccc..bfba20d4c9 100644 --- a/web_src/less/_repository.less +++ b/web_src/less/_repository.less @@ -1,5 +1,4 @@ .repository { - padding-top: 15px; .repo-header { @@ -246,6 +245,12 @@ } &.file.list { + &.blame { + .ui.container:not(.flex) { + width: 98%; + } + } + .repo-description { display: flex; justify-content: space-between; @@ -1646,7 +1651,6 @@ width: 1%; min-width: 50px; user-select: none; - vertical-align: top; span.fold { display: block;