Skip to content

Commit e9cbb30

Browse files
committed
Improve custom avatar UI logic
1 parent 56522a4 commit e9cbb30

File tree

4 files changed

+15
-8
lines changed

4 files changed

+15
-8
lines changed

options/locale/locale_en-US.ini

+1
Original file line numberDiff line numberDiff line change
@@ -468,6 +468,7 @@ keep_activity_private_popup = Makes the activity visible only for you and the ad
468468
lookup_avatar_by_mail = Look Up Avatar by Email Address
469469
federated_avatar_lookup = Federated Avatar Lookup
470470
enable_custom_avatar = Use Custom Avatar
471+
current_avatar = Current avatar
471472
choose_new_avatar = Choose new avatar
472473
update_avatar = Update Avatar
473474
delete_current_avatar = Delete Current Avatar

templates/org/settings/options.tmpl

+1-1
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@
9090
{{.CsrfTokenHtml}}
9191
<div class="df fw ac">
9292
<div class="inline field mb-4">
93-
<label>{{.i18n.Tr "settings.avatar"}}</label>
93+
<label>{{.i18n.Tr "settings.current_avatar"}}</label>
9494
{{avatar .Org 28 "mr-5"}}
9595
</div>
9696
<div class="inline field mb-4">

templates/repo/settings/options.tmpl

+4-2
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@
6060
{{$avatar := (repoAvatar .Repository 28 "mr-5")}}
6161
<div class="df fw ac">
6262
<div class="inline field mb-4">
63-
<label>{{.i18n.Tr "settings.avatar"}}</label>
63+
<label>{{.i18n.Tr "settings.current_avatar"}}</label>
6464
{{if $avatar}}
6565
{{$avatar}}
6666
{{else}}
@@ -75,7 +75,9 @@
7575

7676
<div class="field">
7777
<button class="ui green button">{{$.i18n.Tr "settings.update_avatar"}}</button>
78-
<a class="ui red button delete-post" data-request-url="{{.Link}}/avatar/delete" data-done-url="{{.Link}}">{{$.i18n.Tr "settings.delete_current_avatar"}}</a>
78+
{{if $avatar}}
79+
<a class="ui red button delete-post" data-request-url="{{.Link}}/avatar/delete" data-done-url="{{.Link}}">{{$.i18n.Tr "settings.delete_current_avatar"}}</a>
80+
{{end}}
7981
</div>
8082
</form>
8183

templates/user/settings/profile.tmpl

+9-5
Original file line numberDiff line numberDiff line change
@@ -101,10 +101,12 @@
101101
</div>
102102

103103
<div class="df fw ac">
104-
<div class="inline field mb-4">
105-
<label>{{.i18n.Tr "settings.avatar"}}</label>
106-
{{avatar .SignedUser 28 "mr-5"}}
107-
</div>
104+
{{if .SignedUser.UseCustomAvatar}}
105+
<div class="inline field mb-4">
106+
<label>{{.i18n.Tr "settings.current_avatar"}}</label>
107+
{{avatar .SignedUser 28 "mr-5"}}
108+
</div>
109+
{{end}}
108110
<div class="inline field mb-4">
109111
<label for="avatar">{{.i18n.Tr "settings.choose_new_avatar"}}</label>
110112
<input name="avatar" type="file" >
@@ -113,7 +115,9 @@
113115

114116
<div class="field">
115117
<button class="ui green button">{{$.i18n.Tr "settings.update_avatar"}}</button>
116-
<a class="ui red button delete-post" data-request-url="{{.Link}}/avatar/delete" data-done-url="{{.Link}}">{{$.i18n.Tr "settings.delete_current_avatar"}}</a>
118+
{{if .SignedUser.UseCustomAvatar}}
119+
<a class="ui red button delete-post" data-request-url="{{.Link}}/avatar/delete" data-done-url="{{.Link}}">{{$.i18n.Tr "settings.delete_current_avatar"}}</a>
120+
{{end}}
117121
</div>
118122
</form>
119123
</div>

0 commit comments

Comments
 (0)