Project

General

Profile

Actions

Patch #2285

closed

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

Added by Mischa The Evil about 16 years ago. Updated over 14 years ago.

Status:
Closed
Priority:
Normal
Category:
UI
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:

Description

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.


Files

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

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

Actions
Actions #1

Updated by Markus Knittig about 16 years ago

+1

Actions #2

Updated by Mischa The Evil about 15 years ago

  • Category changed from Issues to UI
Actions #3

Updated by jason axelson about 15 years ago

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

Actions #4

Updated by Benjamin Schindler over 14 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 over 14 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 over 14 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 over 14 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 over 14 years ago

  • Status changed from Resolved to Closed

Merged into 1.0-stable for release in 1.0.2

Actions

Also available in: Atom PDF