Improve commit list/view on mobile (#19712)

- This is a continuation on [the work](https://github.com/go-gitea/gitea/pull/19546) I've done for improving mobile experience on Gitea.
- The current behavior of going trough the commits list is horrible, each individual item gets it's own row and thereby isn't quite compact as it should be on mobile. The commit view's header is in a bit better state, it's quite only that content is overlapping each other.
- This patch fixes those problems. Each row in the commit list table will actually take a row in the UI. The commit view's header has now a better organized way of placing the information.
tokarchuk/v1.17
Gusted 3 years ago committed by GitHub
parent 71ca131582
commit bcf13b670b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 25
      templates/repo/commit_page.tmpl
  2. 4
      templates/repo/commits_list.tmpl
  3. 79
      web_src/less/_repository.less

@ -17,8 +17,8 @@
{{$class = (printf "%s%s" $class " isWarning")}} {{$class = (printf "%s%s" $class " isWarning")}}
{{end}} {{end}}
{{end}} {{end}}
<div class="ui top attached header clearing segment pr {{$class}}"> <div class="ui top attached header clearing segment pr commit-header {{$class}}">
<div class="df mb-4"> <div class="df mb-4 fw">
<h3 class="mb-0 f1"><span class="commit-summary" title="{{.Commit.Summary}}">{{RenderCommitMessage $.Context .Commit.Message $.RepoLink $.Repository.ComposeMetas}}</span>{{template "repo/commit_statuses" dict "Status" .CommitStatus "Statuses" .CommitStatuses "root" $}}</h3> <h3 class="mb-0 f1"><span class="commit-summary" title="{{.Commit.Summary}}">{{RenderCommitMessage $.Context .Commit.Message $.RepoLink $.Repository.ComposeMetas}}</span>{{template "repo/commit_statuses" dict "Status" .CommitStatus "Statuses" .CommitStatuses "root" $}}</h3>
{{if not $.PageIsWiki}} {{if not $.PageIsWiki}}
<div class="ui"> <div class="ui">
@ -143,8 +143,8 @@
<span class="text grey mr-3">{{svg "octicon-tag" 16 "mr-2"}}{{.TagName}}</span> <span class="text grey mr-3">{{svg "octicon-tag" 16 "mr-2"}}{{.TagName}}</span>
{{end}} {{end}}
</div> </div>
<div class="ui attached segment df ac sb py-2 commit-header-row {{$class}}"> <div class="ui attached segment df ac sb py-2 commit-header-row fw {{$class}}">
<div class="df ac"> <div class="df ac author">
{{if .Author}} {{if .Author}}
{{avatar .Author 28 "mr-3"}} {{avatar .Author 28 "mr-3"}}
{{if .Author.FullName}} {{if .Author.FullName}}
@ -171,9 +171,7 @@
<div class="ui horizontal list df ac"> <div class="ui horizontal list df ac">
{{if .Parents}} {{if .Parents}}
<div class="item"> <div class="item">
{{.i18n.Tr "repo.diff.parent"}} <span>{{.i18n.Tr "repo.diff.parent"}}</span>
</div>
<div class="item">
{{range .Parents}} {{range .Parents}}
{{if $.PageIsWiki}} {{if $.PageIsWiki}}
<a class="ui blue sha label" href="{{$.RepoLink}}/wiki/commit/{{PathEscape .}}">{{ShortSha .}}</a> <a class="ui blue sha label" href="{{$.RepoLink}}/wiki/commit/{{PathEscape .}}">{{ShortSha .}}</a>
@ -183,13 +181,14 @@
{{end}} {{end}}
</div> </div>
{{end}} {{end}}
<div class="mobile-only"></div> <div class="item">
<div class="item">{{.i18n.Tr "repo.diff.commit"}}</div> <span>{{.i18n.Tr "repo.diff.commit"}}</span>
<div class="item"><span class="ui blue sha label">{{ShortSha .CommitID}}</span></div> <span class="ui blue sha label">{{ShortSha .CommitID}}</span>
</div>
</div> </div>
</div> </div>
{{if .Commit.Signature}} {{if .Commit.Signature}}
<div class="ui bottom attached message tl df ac sb commit-header-row {{$class}}"> <div class="ui bottom attached message tl df ac sb commit-header-row fw {{$class}}">
<div class="df ac"> <div class="df ac">
{{if .Verification.Verified}} {{if .Verification.Verified}}
{{if ne .Verification.SigningUser.ID 0}} {{if ne .Verification.SigningUser.ID 0}}
@ -204,8 +203,8 @@
{{avatar .Verification.SigningUser 28}} {{avatar .Verification.SigningUser 28}}
<a href="{{.Verification.SigningUser.HomeLink}}"><strong>{{.Verification.SigningUser.GetDisplayName}}</strong></a> <a href="{{.Verification.SigningUser.HomeLink}}"><strong>{{.Verification.SigningUser.GetDisplayName}}</strong></a>
{{else}} {{else}}
<span title="{{.i18n.Tr "gpg.default_key"}}">{{svg "gitea-lock-cog"}}</span> <span title="{{.i18n.Tr "gpg.default_key"}}">{{svg "gitea-lock-cog" 16 "mr-3"}}</span>
<span class="ui text">{{.i18n.Tr "repo.commits.signed_by"}}:</span> <span class="ui text mr-3">{{.i18n.Tr "repo.commits.signed_by"}}:</span>
{{avatarByEmail .Verification.SigningEmail "" 28}} {{avatarByEmail .Verification.SigningEmail "" 28}}
<strong>{{.Verification.SigningUser.GetDisplayName}}</strong> <strong>{{.Verification.SigningUser.GetDisplayName}}</strong>
{{end}} {{end}}

@ -1,5 +1,5 @@
<div class="ui attached table segment"> <div class="ui attached table segment commit-table">
<table class="ui very basic striped fixed table single line" id="commits-table"> <table class="ui very basic striped table unstackable fixed" id="commits-table">
<thead> <thead>
<tr> <tr>
<th class="four wide">{{.i18n.Tr "repo.commits.author"}}</th> <th class="four wide">{{.i18n.Tr "repo.commits.author"}}</th>

@ -3328,4 +3328,83 @@ td.blob-excerpt {
} }
} }
} }
.commit-header-row {
.ui.horizontal.list {
width: 100%;
overflow-x: scroll;
margin-top: 2px;
.item {
align-items: center;
display: flex;
}
}
.author {
padding: 3px 0;
}
}
.commit-header h3 {
flex-basis: auto !important;
margin-bottom: .5rem !important;
}
.commits-table {
flex-direction: column;
.commits-table-left {
align-items: initial !important;
margin-bottom: 6px;
}
.commits-table-right form {
display: flex;
flex-wrap: wrap;
> div:nth-child(1) {
order: 1;
}
> div:nth-child(2) {
order: 3;
margin-left: .5rem;
margin-top: .5rem;
}
> button:nth-child(3) {
order: 2;
margin-left: .25rem;
}
}
}
.commit-table {
overflow-x: scroll;
td.sha,
th.sha {
display: none !important;
}
.commit-list {
span.message-wrapper {
max-width: none;
}
tr td:last-child {
display: block;
width: max-content;
}
td.author {
display: block;
width: calc(100% + .5rem);
}
.copy-commit-sha {
display: none !important;
}
}
}
} }

Loading…
Cancel
Save