Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Ctrl+F does not select the search field #3062

Closed
koppor opened this issue Aug 1, 2017 · 2 comments
Closed

Ctrl+F does not select the search field #3062

koppor opened this issue Aug 1, 2017 · 2 comments
Labels
bug Confirmed bugs or reports that are very likely to be bugs keybinding ui

Comments

@koppor
Copy link
Member

koppor commented Aug 1, 2017

Keywords: shortcut, short cut, keybinding, key binding, hotkey, hot key

JabRef 4.0-dev, Windows 10 10.0 amd64, Java 1.8.0_144

Steps to reproduce:

  1. Focus is on the main table
  2. I press Ctrl+F
  3. Nothing happens

Expected: Focus is on the search field.
@Siedlerchr
Copy link
Member

I debugged a bit, and the action is correclty recognized and the method to requestFocus is called, but somehow the textfield is not focused. Maybe @tobiasdiez has an idea, because he changed that together with the autocompletion.

I even tried to wrap it in a Platform.runLater call, but that did not work

@lenhard lenhard added this to the v4.0 milestone Aug 1, 2017
@Siedlerchr Siedlerchr added bug Confirmed bugs or reports that are very likely to be bugs ui labels Aug 1, 2017
@Siedlerchr Siedlerchr mentioned this issue Aug 1, 2017
Closed
Siedlerchr added a commit that referenced this issue Aug 6, 2017
@Siedlerchr
Merge remote-tracking branch 'upstream/master' into selectFilesDlg
8dca324 
* upstream/master:
  Add warning message if group with same name is already present (#3077)
  Fix #3062: Ctrl + F works again
  Fix del/copy/paste key trigger main table action in search bar (#3070)
  Fix markdown
  Update gradle from 4.0.1 to 4.0.2
  Fix #3045 Update Transformer plugin
  Reimplement MappedList using a backigList (#3069)
  Fix importing preferences after resetting without restarting (#3065)
  Fix some spotbugs issues (#3060)
  Import dialog when fetch (#3025)
  Adapt CircleCI build
  Adapt CI script
  Closes #3027 and updates install4j to v7
@koppor koppor reopened this Jun 3, 2018
@koppor koppor removed this from the v4.0 milestone Jun 3, 2018
@koppor
Copy link
Member Author

koppor commented Jun 3, 2018

Happens again in the maintable-beta branch. As discussed in the devcall, I should open issues for issues with that branch (which will become the master branch).

@tobiasdiez tobiasdiez mentioned this issue Jun 3, 2018
Merged
35 tasks
@tobiasdiez tobiasdiez mentioned this issue Jul 8, 2018
Closed
tobiasdiez added a commit that referenced this issue Jul 8, 2018
@tobiasdiez
Fix #3062: Ctrl + F selects search field again
ae30273
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Confirmed bugs or reports that are very likely to be bugs keybinding ui
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@Siedlerchr @koppor @lenhard @tobiasdiez