Project

General

Profile

Actions
Copy link

Defect #22023

closed

Issue id input should get focus after adding related issue

Added by Jim Poserina over 8 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Go MAEDA
Category:
UI
Target version:
3.3.8
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Affected version:
2.4.0

Description

Current behavior when adding multiple related issues:

  • Click Add
  • Issue # text input has focus
  • Enter issue number, press Enter key
  • The add related issue subform stays open, but the Issue # input has lost focus

To add another related issue, you must either:

  • Tab 5 times (to cycle through the Add link, the link to the related issue, the Delete relation link, the relation type drop-down, and then the Issue # input). Then to add a third issue, you must tab 7 times. For a fourth, 9 times, and so on.
  • Move the mouse and re-click the text input.
  • Click Add to close the subform and then Add to reopen it, whereupon the Issue # input receives focus.

The UI anticipates that you may need to enter multiple related issues as the subform stays open after each addition. Thus after a related issue is added the Issue # input should retain focus.

Actions
Copy link
 #1

Updated by Jim Poserina over 6 years ago

Ping

Actions
Copy link
 #2

Updated by Mizuki ISHIKAWA over 6 years ago

This feature can be realized by the following small change.

diff --git a/app/views/issue_relations/create.js.erb b/app/views/issue_relations/create.js.erb
index 0c2e036cc5..1f296e66d4 100644
--- a/app/views/issue_relations/create.js.erb
+++ b/app/views/issue_relations/create.js.erb
@@ -2,6 +2,6 @@ $('#relations').html('<%= escape_javascript(render :partial => 'issues/relations
 <% if @relation.errors.empty? %>
   $('#relation_delay').val('');
   $('#relation_issue_to_id').val('');
-  $('#relation_issue_to_id').focus();
 <% end %>
 $('#new-relation-form').show();
+$('#relation_issue_to_id').focus();
\ No newline at end of file
Actions
Copy link
 #3

Updated by Go MAEDA over 6 years ago

  • Target version set to 3.3.8

The patch posted by Mizuki works fine. Setting target version to 3.3.8.

The current code of Redmine sets focus to the text box in source:tags/3.4.5/app/views/issue_relations/create.js.erb#L5 but the focus will be cleared by $('#new-relation-form').show(); in source:tags/3.4.5/app/views/issue_relations/create.js.erb#L7.

The original code tries to set the focus only when no error occurs, but the proposed patch also sets the focus when an error occurs. I think the behavior of the patch is more natural.

Actions
Copy link
 #4

Updated by Go MAEDA over 6 years ago

  • Tracker changed from Feature to Defect
  • Subject changed from Have the Issue # input retain focus after adding related issue to Issue ID input should get focus after adding related issue
  • Affected version set to 2.4.0

This bug seems to be introduced in r11610 (Redmine 2.4.0).

Actions
Copy link
 #5

Updated by Go MAEDA over 6 years ago

  • Subject changed from Issue ID input should get focus after adding related issue to Issue id input should get focus after adding related issue
  • Status changed from New to Closed
  • Assignee set to Go MAEDA

Committed to the trunk and stable branches. Thank you all for reporting and fixing this issue.

Actions
Copy link

Also available in: Atom PDF