Browse Source

Do not show the "Sign up for free" button in issue tracker (#3408)

* Do not show the "Sign up for free" button in issue tracker

The "Sign in to comment" link is good enough and will correctly
show or not show the "Sign Up" button link for those not having
an account already.

Fixes #3407 (link to nowhere when registration is disabled)

* Move html from translation to template

* Remove extra space in `{{ .SignInLink }}`.
master
Sandro Santilli 无闻 8 years ago
parent
commit
eb1bfe0e59
2 changed files with 2 additions and 4 deletions
  1. +1
    -2
      conf/locale/locale_en-US.ini
  2. +1
    -2
      templates/repo/issue/view_content.tmpl

+ 1
- 2
conf/locale/locale_en-US.ini View File

@@ -485,8 +485,7 @@ issues.commit_ref_at = `referenced this issue from a commit <a id="%[1]s" href="
issues.poster = Poster issues.poster = Poster
issues.collaborator = Collaborator issues.collaborator = Collaborator
issues.owner = Owner issues.owner = Owner
issues.sign_up_for_free = Sign up for free
issues.sign_in_require_desc = to join this conversation. Already have an account? <a href="%s">Sign in to comment</a>
issues.sign_in_require_desc = Sign in to comment
issues.edit = Edit issues.edit = Edit
issues.cancel = Cancel issues.cancel = Cancel
issues.save = Save issues.save = Save


+ 1
- 2
templates/repo/issue/view_content.tmpl View File

@@ -227,8 +227,7 @@
</div> </div>
{{else}} {{else}}
<div class="ui warning message"> <div class="ui warning message">
<a href="{{AppSubUrl}}/user/sign_up" class="ui green button">{{.i18n.Tr "repo.issues.sign_up_for_free"}}</a>
{{.i18n.Tr "repo.issues.sign_in_require_desc" .SignInLink | Safe}}
<a href="{{.SignInLink}}">{{.i18n.Tr "repo.issues.sign_in_require_desc" | Safe}}</a>
</div> </div>
{{end}} {{end}}
</ui> </ui>


Loading…
Cancel
Save