Feature #15275
Improve usage of label "button_update"
Status: | Closed | Start date: | ||
---|---|---|---|---|
Priority: | Low | Due date: | ||
Assignee: | % Done: | 0% | ||
Category: | Translations | |||
Target version: | 2.5.0 | |||
Resolution: | Fixed |
Description
Hi, today I discovered an improper german translation of button_update
:
button_update: Bearbeiten
should better be
button_update: Aktualisieren
While the first has the meaning of "edit", the second corresponds to "update by applying my changes". Unfortunately, button_update
seems also to be used for editing issues:
$ grep -rn button_update app/ app/views/issues/_action_menu.html.erb:2:<%= link_to l(:button_update), edit_issue_path(@issue), :onclick => 'showAndScrollTo("update", "issue_notes"); return false;', :class => 'icon ico n-edit', :accesskey => accesskey(:edit) if @issue.editable? %> app/views/issues/show.html.erb:131: <h3><%= l(:button_update) %></h3> app/views/repositories/committers.html.erb:39:<p><%= submit_tag(l(:button_update)) %></p>
Therefore, I would suggest to change the translation and change button_update
to button_edit
for the first two occurrences. It is fine for the third, I think.
What do you think? Comments welcome!
Related issues
Associated revisions
Use :button_save instead of :button_update on committers form (#15275).
Use :button_edit instead of :button_update for editing issues (#15275).
History
#1
Updated by Björn Peemöller over 7 years ago
Well, another solution may also be to change button_update
to button_save
in app/views/repositories/committers.html.erb:39
, but I think the other change may also be worth it.
#2
Updated by Jean-Philippe Lang over 7 years ago
- Status changed from New to Closed
- Assignee set to Jean-Philippe Lang
- Target version set to 2.5.0
- Resolution set to Fixed
Changes done.
#3
Updated by Björn Peemöller over 7 years ago
Cool, thanks!
#4
Updated by Stanislav German-Evtushenko almost 7 years ago
And now we have a weird way to add a comment by the "Edit" button. I'm sure we will get used to it but it doesn't seem logical.
#5
Updated by Go MAEDA over 6 years ago
- Related to Defect #7856: Change "Update" to "Update/Reply" on issue#show added