feat(ui/migrations): add placeholder to clarify empty description behavior (#7373)

Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/7373
Reviewed-by: 0ko <0ko@noreply.codeberg.org>
Co-authored-by: Mat <m.falkowski@dotmat.de>
Co-committed-by: Mat <m.falkowski@dotmat.de>
This commit is contained in:
Mat 2025-03-29 10:21:07 +00:00 committed by 0ko
parent bf8bdf12df
commit 8df8381f51
8 changed files with 14 additions and 13 deletions

View file

@ -1245,6 +1245,7 @@ migrate_items_pullrequests = Pull requests
migrate_items_merge_requests = Merge requests
migrate_items_releases = Releases
migrate_repo = Migrate repository
migrate.repo_desc_helper = Leave empty to import existing description
migrate.clone_address = Migrate / Clone from URL
migrate.clone_address_desc = The HTTP(S) or Git "clone" URL of an existing repository
migrate.github_token_desc = You can put one or more tokens with comma separated here to make migrating faster because of GitHub API rate limit. WARN: Abusing this feature may violate the service provider's policy and lead to account blocking.

View file

@ -109,7 +109,7 @@
</div>
<div class="inline field {{if .Err_Description}}error{{end}}">
<label for="description">{{ctx.Locale.Tr "repo.repo_desc"}}</label>
<textarea id="description" name="description" maxlength="2048">{{.description}}</textarea>
<textarea id="description" name="description" maxlength="2048" placeholder="{{ctx.Locale.Tr "repo.migrate.repo_desc_helper"}}">{{.description}}</textarea>
</div>
<div class="inline field">

View file

@ -121,7 +121,7 @@
</div>
<div class="inline field {{if .Err_Description}}error{{end}}">
<label for="description">{{ctx.Locale.Tr "repo.repo_desc"}}</label>
<textarea id="description" name="description" maxlength="2048">{{.description}}</textarea>
<textarea id="description" name="description" maxlength="2048" placeholder="{{ctx.Locale.Tr "repo.migrate.repo_desc_helper"}}">{{.description}}</textarea>
</div>
<div class="inline field">

View file

@ -123,7 +123,7 @@
</div>
<div class="inline field {{if .Err_Description}}error{{end}}">
<label for="description">{{ctx.Locale.Tr "repo.repo_desc"}}</label>
<textarea id="description" name="description" maxlength="2048">{{.description}}</textarea>
<textarea id="description" name="description" maxlength="2048" placeholder="{{ctx.Locale.Tr "repo.migrate.repo_desc_helper"}}">{{.description}}</textarea>
</div>
<div class="inline field">

View file

@ -120,7 +120,7 @@
</div>
<div class="inline field {{if .Err_Description}}error{{end}}">
<label for="description">{{ctx.Locale.Tr "repo.repo_desc"}}</label>
<textarea id="description" name="description" maxlength="2048">{{.description}}</textarea>
<textarea id="description" name="description" maxlength="2048" placeholder="{{ctx.Locale.Tr "repo.migrate.repo_desc_helper"}}">{{.description}}</textarea>
</div>
<div class="inline field">

View file

@ -120,7 +120,7 @@
</div>
<div class="inline field {{if .Err_Description}}error{{end}}">
<label for="description">{{ctx.Locale.Tr "repo.repo_desc"}}</label>
<textarea id="description" name="description" maxlength="2048">{{.description}}</textarea>
<textarea id="description" name="description" maxlength="2048" placeholder="{{ctx.Locale.Tr "repo.migrate.repo_desc_helper"}}">{{.description}}</textarea>
</div>
<div class="inline field">

View file

@ -109,7 +109,7 @@
</div>
<div class="inline field {{if .Err_Description}}error{{end}}">
<label for="description">{{ctx.Locale.Tr "repo.repo_desc"}}</label>
<textarea id="description" name="description" maxlength="2048">{{.description}}</textarea>
<textarea id="description" name="description" maxlength="2048" placeholder="{{ctx.Locale.Tr "repo.migrate.repo_desc_helper"}}">{{.description}}</textarea>
</div>
<div class="inline field">

View file

@ -30,28 +30,28 @@ func TestRepoMigrationUI(t *testing.T) {
2: {
"GitHub",
[]string{"issues", "pull_requests", "labels", "milestones", "releases"},
false,
true,
},
3: {
"Gitea",
[]string{"issues", "pull_requests", "labels", "milestones", "releases"},
false,
true,
},
4: {
"GitLab",
// Note: the checkbox "Merge requests" has name "pull_requests"
[]string{"issues", "pull_requests", "labels", "milestones", "releases"},
false,
true,
},
5: {
"Gogs",
[]string{"issues", "labels", "milestones"},
false,
true,
},
6: {
"OneDev",
[]string{"issues", "pull_requests", "labels", "milestones"},
false,
true,
},
7: {
"GitBucket",
@ -62,12 +62,12 @@ func TestRepoMigrationUI(t *testing.T) {
"Codebase",
// Note: the checkbox "Merge requests" has name "pull_requests"
[]string{"issues", "pull_requests", "labels", "milestones"},
false,
true,
},
9: {
"Forgejo",
[]string{"issues", "pull_requests", "labels", "milestones", "releases"},
false,
true,
},
}