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.

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

No way to search from inside a note on Android #2389

Open
dbogdanov opened this issue Jan 28, 2020 · 23 comments
Open

No way to search from inside a note on Android #2389

dbogdanov opened this issue Jan 28, 2020 · 23 comments
Labels
backlog We'll get to it... eventually... enhancement Feature requests and code enhancements mobile All mobile platforms

Comments

@dbogdanov
Copy link

There is no way to search for text in a note from inside the note view on Android. Desktop versions of Joplin provide this functionality, but not on Android. There are previous issues describing this problem that were automatically closed (see #382).

As I understand, b189814 adds an ability to highlight the text within notes, so one can search from the main note search menu, but this is useless and using this function is not intuitive when a user is already inside a note.

The open menu on the screenshot should contain another option to "Search in this note".

188

Environment

Joplin version: 1.0.316
Platform: Android

@dbogdanov dbogdanov added the bug It's a bug label Jan 28, 2020
@amitsin6h
Copy link
Contributor

I would like to work on this @CalebJohn @dbogdanov can you assign me

@amitsin6h
Copy link
Contributor

I am working on this issue 👨‍💻

@amitsin6h
Copy link
Contributor

Mentors please review my PR #2423 and let me know if it requires any changes.
@tessus @CalebJohn @laurent22

@amitsin6h
Copy link
Contributor

@laurent22 can you please help me with this, I tried to forceUpdate the component based on props changes but does not work, also I created ref for the webview and thought to refresh whenever search keyword props changes but still the highlight was not working do you have a suggestion for me

@stale
Copy link

stale bot commented Apr 15, 2020

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? 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.

@stale stale bot added the stale An issue that hasn't been active for a while... label Apr 15, 2020
@dbogdanov
Copy link
Author

Not stale.

@stale stale bot removed the stale An issue that hasn't been active for a while... label Apr 15, 2020
@stale
Copy link

stale bot commented May 30, 2020

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? 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.

@stale stale bot added the stale An issue that hasn't been active for a while... label May 30, 2020
@dbogdanov
Copy link
Author

Not stale.

@stale stale bot removed the stale An issue that hasn't been active for a while... label Jun 2, 2020
@laurent22 laurent22 added backlog We'll get to it... eventually... enhancement Feature requests and code enhancements mobile All mobile platforms and removed bug It's a bug labels Jun 2, 2020
@cosminribo
Copy link

Searching in long notes is quite essential, both in the Android and desktop client.

@marcolaux
Copy link

@amitsin6h is there new progress on this feature?

@arpitg24
Copy link

Is anyone currently working on this issue?

@tessus
Copy link
Collaborator

tessus commented Feb 24, 2021

@arpitg24 not that I'm aware of.

@arpitg24
Copy link

I would like to work on this. @tessus can you please assign this issue to me?

@laurent22
Copy link
Owner

We don't assign issues. If you can work on it then please do so.

@arpitg24
Copy link

arpitg24 commented Feb 28, 2021

I have added a 'Search in this note' button to the side menu, I'm trying to figure out a way to get text input from the user. Can anyone help me with this, I'm planning to use some sort of custom alert to get input from the user?

@Pilastr
Copy link

Pilastr commented Mar 24, 2021

@arpitg24 24 days on, is this effort ongoing or DOA?

@arpitg24
Copy link

arpitg24 commented Mar 25, 2021

@Pilastr I dont think i'd be able to finish it since it would require a significant amount of code writing.

@TheSYNcoder
Copy link

@dbogdanov @laurent22 I would like to work on this issue. Is anyone assigned yet or I can work on it?

@laurent22
Copy link
Owner

It's not on GSoC, and I'm not certain it can be done due to the buggy text input control (broken selection API, and I doubt scrolling is much better).

@TheSYNcoder
Copy link

It's not on GSoC, and I'm not certain it can be done due to the buggy text input control (broken selection API, and I doubt scrolling is much better).

@laurent22 Okay thanks. I will try to look into the codebase if i can do something. Can you suggest some bugs that i can solve that can be related to GSoC. Thanks.

@personalizedrefrigerator
Copy link
Collaborator

Looking at the CodeMirror 6 basic-setup plugin and the CodeMirror 6 documentation, it looks like we can show/hide the existing search panel if we enable it! This would only apply to the beta editor, though...

@laurent22
Copy link
Owner

Looking at the CodeMirror 6 basic-setup plugin and the CodeMirror 6 documentation, it looks like we can show/hide the existing search panel if we enable it! This would only apply to the beta editor, though...

Indeed that would be a great solution if we can simply rely on the built-in search.

@guy-rouillier
Copy link

I want to add my vote for being able to search while viewing an individual note. I found a discussion that searching within a single note is available if I edit the note. Indeed, that does work. But this is not ideal. I spend more time viewing notes than I do editing them (thankfully.) So being able to search a note in view mode would be very helpful.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog We'll get to it... eventually... enhancement Feature requests and code enhancements mobile All mobile platforms
Projects
None yet
Development

No branches or pull requests