Skip to content

Commit a2d7999

Browse files
Insert ui divider directly in templates instead of from inside heatmap vue component (#11508)
* ui: remove the divider between heatmap and feeds in the dashboard view * this commit reverses 6ccd19e Signed-off-by: surtur <[email protected]> * add a divider to the default user dashboard view * another one-line change, this time as a direct complement of 2bfc716 to implement a divider of sorts (as per 6ccd19e originally) Signed-off-by: surtur <[email protected]> * removed the divider in heatmap.tmpl * the separator is added by partial templates as needed Signed-off-by: surtur <[email protected]> * load proper dashboard template code in profile.tmpl * as discussed in Discord * includes a divider of its own Signed-off-by: surtur <[email protected]>
1 parent 50a9727 commit a2d7999

File tree

4 files changed

+5
-12
lines changed

4 files changed

+5
-12
lines changed

templates/user/dashboard/dashboard.tmpl

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
<div class="ui container ten wide column">
88
{{if .EnableHeatmap}}
99
{{template "user/dashboard/heatmap" .}}
10+
<div class="ui divider"></div>
1011
{{end}}
1112
{{template "user/dashboard/feeds" .}}
1213
</div>

templates/user/dashboard/heatmap.tmpl

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,5 +4,4 @@
44
<div class="ui active centered inline indeterminate text loader" id="loading-heatmap">{{.i18n.Tr "user.heatmap.loading"}}</div>
55
</div>
66
</activity-heatmap>
7-
<div class="ui divider"></div>
87
</div>

templates/user/profile.tmpl

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -104,16 +104,10 @@
104104
</div>
105105

106106
{{if eq .TabName "activity"}}
107-
{{if .EnableHeatmap}}
108-
<div id="user-heatmap" style="padding-right: 40px">
109-
<activity-heatmap :locale="locale" :suburl="suburl" :user="heatmapUser">
110-
<div slot="loading">
111-
<div class="ui active centered inline indeterminate text loader" id="loading-heatmap">{{.i18n.Tr "user.heatmap.loading"}}</div>
112-
</div>
113-
</activity-heatmap>
114-
</div>
115-
<div class="ui divider"></div>
116-
{{end}}
107+
{{if .EnableHeatmap}}
108+
{{template "user/dashboard/heatmap" .}}
109+
<div class="ui divider"></div>
110+
{{end}}
117111
<div class="feeds">
118112
{{template "user/dashboard/feeds" .}}
119113
</div>

web_src/js/components/ActivityHeatmap.vue

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77
{{ totalContributions }} total contributions in the last 12 months
88
</h4>
99
<calendar-heatmap v-show="!isLoading" :locale="locale" :no-data-text="locale.no_contributions" :tooltip-unit="locale.contributions" :end-date="endDate" :values="values" :range-color="colorRange"/>
10-
<div class="ui divider"></div>
1110
</div>
1211
</template>
1312

0 commit comments

Comments
 (0)