Skip to content

RSS icon fixes #24476

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 16 commits into from
May 10, 2023
Merged

RSS icon fixes #24476

Show file tree
Hide file tree
Changes from 11 commits
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
8f5a631
RSS icon fixes
silverwind May 1, 2023
032cb7f
move to features
silverwind May 1, 2023
d74a638
adjust selector
silverwind May 1, 2023
6d5d402
move sub_menu_release_tag to release_tag_header and add a div for it
silverwind May 7, 2023
7697409
make div name match filename
silverwind May 7, 2023
fc63af0
remove unused classes
silverwind May 7, 2023
2f4932b
undo index changes
silverwind May 7, 2023
25cf626
remove fallback rendering of repo/sub_menu on release/tag page
silverwind May 8, 2023
c5b0954
re-add repo-summary to release page with full width
silverwind May 9, 2023
9e00958
Merge branch 'main' into rss-icon
wxiaoguang May 10, 2023
50cf809
Merge branch 'main' into rss-icon
wxiaoguang May 10, 2023
148d0cd
refator
wxiaoguang May 10, 2023
69fc1cd
fine tune
wxiaoguang May 10, 2023
ff86cb8
Update templates/repo/tag/list.tmpl
wxiaoguang May 10, 2023
e936081
move repo rss icon after labels
silverwind May 10, 2023
8bb37f7
Merge branch 'main' into rss-icon
GiteaBot May 10, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions templates/org/home.tmpl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,15 +3,15 @@
<div class="ui container gt-df">
{{avatar $.Context .Org 140 "org-avatar"}}
<div id="org-info">
<div class="ui header">
<div class="ui header gt-df gt-ac gt-fw">
{{.Org.DisplayName}}
{{if .EnableFeed}}
<a href="{{.Org.HomeLink}}.rss"><i class="ui grey icon gt-ml-3" data-tooltip-content="{{.locale.Tr "rss_feed"}}">{{svg "octicon-rss" 36}}</i></a>
{{end}}
<span class="org-visibility">
<span class="org-visibility gt-df gt-ml-3">
{{if .Org.Visibility.IsLimited}}<div class="ui large basic horizontal label">{{.locale.Tr "org.settings.visibility.limited_shortname"}}</div>{{end}}
{{if .Org.Visibility.IsPrivate}}<div class="ui large basic horizontal label">{{.locale.Tr "org.settings.visibility.private_shortname"}}</div>{{end}}
</span>
{{if .EnableFeed}}
<a href="{{.Org.HomeLink}}.rss" class="rss-icon" data-tooltip-content="{{.locale.Tr "rss_feed"}}">{{svg "octicon-rss" 24}}</a>
{{end}}
</div>
{{if $.RenderedDescription}}<div class="render-content markup">{{$.RenderedDescription|Str2html}}</div>{{end}}
<div class="text grey meta">
Expand Down
2 changes: 1 addition & 1 deletion templates/repo/header.tmpl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
<div class="gt-mx-2">/</div>
<a href="{{$.RepoLink}}">{{.Name}}</a>
{{if $.EnableFeed}}
<a href="{{$.RepoLink}}.rss"><i class="ui grey icon gt-ml-3" data-tooltip-content="{{$.locale.Tr "rss_feed"}}">{{svg "octicon-rss" 18}}</i></a>
<a href="{{$.RepoLink}}.rss" class="rss-icon" data-tooltip-content="{{$.locale.Tr "rss_feed"}}">{{svg "octicon-rss" 18}}</a>
{{end}}
<div class="labels gt-df gt-ac gt-fw">
{{if .IsTemplate}}
Expand Down
9 changes: 1 addition & 8 deletions templates/repo/release/list.tmpl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,14 +3,7 @@
{{template "repo/header" .}}
<div class="ui container">
{{template "base/alert" .}}
{{template "repo/sub_menu_release_tag" .}}

{{if .CanCreateRelease}}
<a class="ui right small green button" href="{{$.RepoLink}}/releases/new">
{{.locale.Tr "repo.release.new_release"}}
</a>
{{end}}

{{template "repo/release_tag_header" .}}
<ul id="release-list">
{{range $idx, $release := .Releases}}
<li class="ui grid">
Expand Down
26 changes: 26 additions & 0 deletions templates/repo/release_tag_header.tmpl
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,26 @@
{{$canReadReleases := $.Permission.CanRead $.UnitTypeReleases}}
{{$canReadCode := $.Permission.CanRead $.UnitTypeCode}}
<div class="release-tag-header gt-df gt-ac">
<div class="release-tag-header-left gt-df gt-ac gt-f1">
{{if $canReadReleases}}
<h2 class="ui compact small menu header small-menu-items">
<a class="{{if .PageIsReleaseList}}active {{end}}item" href="{{.RepoLink}}/releases">{{.locale.Tr "repo.release.releases"}}</a>
{{if $canReadCode}}
<a class="{{if .PageIsTagList}}active {{end}}item" href="{{.RepoLink}}/tags">{{.locale.Tr "repo.release.tags"}}</a>
{{end}}
</h2>
{{if .EnableFeed}}
<a class="rss-icon" href="{{.RepoLink}}/{{if .PageIsTagList}}tags{{else}}releases{{end}}.rss" data-tooltip-content="{{.locale.Tr "rss_feed"}}">{{svg "octicon-rss" 18}}</a>
{{end}}
{{else if $canReadCode}}
{{template "repo/sub_menu" .}}
{{end}}
</div>
<div class="release-tag-header-right">
{{if and (not .PageIsTagList) .CanCreateRelease}}
<a class="ui right small green button" href="{{$.RepoLink}}/releases/new">
{{.locale.Tr "repo.release.new_release"}}
</a>
{{end}}
</div>
</div>
17 changes: 0 additions & 17 deletions templates/repo/sub_menu_release_tag.tmpl
View file
Open in desktop

This file was deleted.

10 changes: 2 additions & 8 deletions templates/repo/tag/list.tmpl
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,21 +1,15 @@
{{template "base/head" .}}

<div role="main" aria-label="{{.Title}}" class="page-content repository tags">
{{template "repo/header" .}}
<div class="ui container">
{{template "base/alert" .}}
{{template "repo/sub_menu_release_tag" .}}

<div class="ui divider"></div>

<h4 class="ui top attached header">
{{template "repo/release_tag_header" .}}
<h4 class="ui top attached header gt-mt-4">
<div class="five wide column gt-df gt-ac">
{{svg "octicon-tag" 16 "gt-mr-2"}}{{.locale.Tr "repo.release.tags"}}
</div>
</h4>

{{$canReadReleases := $.Permission.CanRead $.UnitTypeReleases}}

<div class="ui attached table segment">
<table class="ui very basic striped fixed table single line" id="tags-table">
<tbody class="tag-list">
Expand Down
10 changes: 10 additions & 0 deletions web_src/css/base.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2573,6 +2573,16 @@ a.ui.basic.label:hover {
line-height: .67em;
}

.rss-icon {
display: inline-flex;
margin-left: 8px;
color: var(--color-text-light-1);
}

.organization.profile .rss-icon {
margin-left: 0;
}

table th[data-sortt-asc]:hover,
table th[data-sortt-desc]:hover {
background: rgba(0, 0, 0, 0.1) !important;
Expand Down
5 changes: 0 additions & 5 deletions web_src/css/organization.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -112,11 +112,6 @@
margin-bottom: 0;
}

.organization.profile #org-info .ui.header .org-visibility .label {
margin-left: 5px;
margin-top: 2px;
}

.organization.profile #org-info .desc {
font-size: 16px;
margin-bottom: 10px;
Expand Down
6 changes: 6 additions & 0 deletions web_src/css/repository/release-tag.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -132,3 +132,9 @@
.repository.new.release .field .attachment_edit {
width: 450px !important;
}

/* stretch summary when rendered in releases page in case the
repo has the releases tab disabled */
.release-tag-header .repository-summary {
width: 100%;
}
2 changes: 1 addition & 1 deletion web_src/js/components/RepoBranchTagSelector.vue
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@
<div class="scrolling menu" ref="scrollContainer">
<div v-for="(item, index) in filteredItems" :key="item.name" class="item" :class="{selected: item.selected, active: active === index}" @click="selectItem(item)" :ref="'listItem' + index">
{{ item.name }}
<a v-if="enableFeed && mode === 'branches'" role="button" class="ui compact muted right" :href="rssURLPrefix + item.url" target="_blank" @click.stop>
<a v-if="enableFeed && mode === 'branches'" role="button" class="ui compact right rss-icon" :href="rssURLPrefix + item.url" target="_blank" @click.stop>
<svg-icon name="octicon-rss" :size="14"/>
</a>
</div>
Expand Down