joplin icon indicating copy to clipboard operation
joplin copied to clipboard

Arrow keys cannot be used to navigate search results

Open AdamLearns opened this issue 1 year ago • 12 comments

Operating system

macOS

Joplin version

Joplin 2.13.9 (prod, darwin)

Desktop version info

Joplin 2.13.9 (prod, darwin)

Client ID: 76122f1042cc4756b47308502b4b719d
Sync Version: 3
Profile Version: 44
Keychain Supported: No

Revision: 1bbec44

Current behaviour

  1. Start a search by pressing ⌘⇧F.
  2. Type a search query that appears in multiple notes (e.g. "the").
  3. Try pressing the down arrow on your keyboard to go to the search results.

Here's a video recording of me trying this out.

Expected behaviour

I expected the focus to go down to the search results and optionally to preview the selected note (although I don't mind pressing enter at that point either).

I did see issue #6352, but it seems to have been closed with no resolution.

Logs

No response

AdamLearns avatar Dec 18 '23 23:12 AdamLearns

Hey there, it looks like there has been no activity on this issue recently. Has the issue been fixed, or does it still require the community's attention? If you require support or are requesting an enhancement or feature then please create a topic on the Joplin forum. This issue may be closed if no further activity occurs. You may comment on the issue and I will leave it open. Thank you for your contributions.

github-actions[bot] avatar Jan 27 '24 16:01 github-actions[bot]

I downloaded the latest version to test this (2.13.15), and it's still an issue. I'm no longer using Joplin, so I probably won't be commenting in the future to keep this issue alive. 😢

AdamLearns avatar Jan 28 '24 22:01 AdamLearns

For reference, here are a few relevant files:

Before working on this issue we should define how it should work. I've never seen an app move to the search results by pressing the down arrow or even Enter. Tab maybe? Because it won't be documented it needs to be intuitive

laurent22 avatar Feb 27 '24 10:02 laurent22

I guess Enter makes the most sense because that's the traditional approach towards search i.e. when we talk about search engines. Enter takes us to the search results.

njmulsqb avatar Feb 27 '24 12:02 njmulsqb

I guess Enter makes the most sense because that's the traditional approach towards search i.e. when we talk about search engines. Enter takes us to the search results.

But here the results are displayed in realtime

laurent22 avatar Feb 27 '24 12:02 laurent22

But here the results are displayed in realtime

Right. I hadn't given it much thought just that I saw this issue open labelled "good-first-issue" I assumed this functionality is what Joplin is looking to implement.

njmulsqb avatar Feb 27 '24 12:02 njmulsqb

Yes, I think the problem is that I misread the task. I thought the suggestion was to use arrow keys to go from one result to another, but that's already implemented. Having a key to jump to the results from the search field is a reasonable feature to add too, but we need to clearly specify what we want

laurent22 avatar Feb 27 '24 13:02 laurent22

Hello, I found this issue since it has good first issue label, I also read the discussion.

By chance, I noticed discource has the same feature (not quite), you press enter, query results, then you're able to navigate results if you press arrow-down. You can also navigate to the search bar using same arrows.

https://github.com/laurent22/joplin/assets/12874707/320b5bcb-2d2c-4b7b-b668-2c0b77d41d50

Edit: just cropped the video to make it more presentable.

G0maa avatar Mar 16 '24 08:03 G0maa

hey i have just noticed this issue and I think I can solve this , is the issue still open to work ?

Varun-2538 avatar Mar 27 '24 09:03 Varun-2538

Can we implement it using refs, in the NoteListWrapper.tsx file, we can pass a ref in the NoteList2 component and set it there and we also pass it in the SearchBar component via NoteListControls component and we can shift the focus on the press of arrow down key in the SearchBar component.

ahsanfarooq210 avatar Mar 29 '24 21:03 ahsanfarooq210

I would love to claim this. I might solve this issue.

zeul22 avatar May 21 '24 10:05 zeul22

Hi, I'm interested in working on this issue and would love to contribute to the project. Could you please assign it to me? @laurent22 @personalizedrefrigerator

abhinav26966 avatar Jul 10 '24 07:07 abhinav26966