Patch #2285


issue-refinement: pressing enter should result to an "apply"

Added by Mischa The Evil almost 15 years ago. Updated about 13 years ago.

Target version:
Start date:
Due date:
% Done:


Estimated time:


This is just a plain copy-and-paste of issue 334 on the Typo3-forge:

Sometimes I have fields in the issue-filters form, where i can enter text. When i press enter, nothing happens. I always have to click "Apply".
It would be much faster if a enter would result into an apply.
It is just a nice-to-have feature.

This patch is also a straight one-on-one copy of the solution implemented by the Typo3-forge maintainers, rebased against r2088.


issue_refinement-enter-result_apply-r2088.diff (2.17 KB) issue_refinement-enter-result_apply-r2088.diff Patch against r2088 Mischa The Evil, 2008-12-06 00:19
2285_git.diff (1.36 KB) 2285_git.diff Jean-Baptiste Barth, 2010-09-05 20:30

Related issues

Has duplicate Redmine - Feature #4034: apply the filter on issues by pressing Return keyClosed2009-10-15

Has duplicate Redmine - Defect #6294: Pressing "Enter" key when filling a filter triggers "Save" actionClosed2010-09-05

Actions #1

Updated by Markus Knittig almost 15 years ago


Actions #2

Updated by Mischa The Evil almost 14 years ago

  • Category changed from Issues to UI
Actions #3

Updated by jason axelson almost 14 years ago

So is there a chance of this being merged into the main codebase?

Actions #4

Updated by Benjamin Schindler about 13 years ago

+1 here. In my installation, pressing enter brings me to save queries, which was in my case mostly not what I wanted. Apply would be a lot more intuitive imho

Actions #5

Updated by Jean-Baptiste Barth about 13 years ago

  • Status changed from New to 7
  • Assignee set to Jean-Baptiste Barth
  • Target version set to 1.0.2

Works fine for me on Firefox 3.6, but not on Chrome 6. Which browser have you tested ?

I'll try to see if it works better with a more prototype-based code.

Actions #6

Updated by Jean-Baptiste Barth about 13 years ago

Here's a working solution with Prototype, it avoids treating each navigator separately. Can someone confirm it works under IE ? (I don't have windows at home I can't test that...). Don't hesitate to tell me if there's something bad, I'm not a JS specialist at all.

Actions #7

Updated by Jean-Baptiste Barth about 13 years ago

  • Status changed from 7 to Resolved
  • % Done changed from 0 to 100

Applied in r4093. Don't hesitate to reopen if you encounter navigator specific issues.

Actions #8

Updated by Eric Davis about 13 years ago

  • Status changed from Resolved to Closed

Merged into 1.0-stable for release in 1.0.2


Also available in: Atom PDF