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

Search Directory + Search Content is excluding mjs/js files #2511

Open
4 tasks done
hmmuaa opened this issue Jan 21, 2025 · 12 comments
Open
4 tasks done

Search Directory + Search Content is excluding mjs/js files #2511

hmmuaa opened this issue Jan 21, 2025 · 12 comments

Comments

@hmmuaa
Copy link

hmmuaa commented Jan 21, 2025

⚠️ This issue respects the following points: ⚠️

  • This is a enhancement/feature request. Not a bug or question.
  • The topic is not already reported at Issues. (I've searched it).
  • Markor is up to date. See Releases for the latest version. Updates are available from F-Droid and GitHub.
  • The wanted feature/enhancement is not present in the latest development version (git master). (Please download and try the test version of Markor, named Marder. Don't worry; Markor and Marder appear as completely separate applications. You can install both side-by-side, and Markor's settings are not touched. If your desired feature is present, you don't need to open this issue. The change will be part of the next Markor update.)

Description

  1. I've turned on “Always open with this APP” for .mjs
  2. Tried modified the file ext name to md, n/got it correctly searched
  3. but if I keep the .mjs ext, can't get the content b/searched by Search Directory

I also tried rename .mjs to .js, still not work

I've tried several, such as finding 'import'
it may got results from many files, but indeed nothing found

-the dir could found here https://github.com/hmmuaa/blog/tree/main/js/lab/lib
-my searching dir is under my Notebook dir, dunno if is this necessary

Information

Android version:
Device:
App Version: 2.14(157

Source

Google Play

Format / File type

Something else (-> Additional info)

Additional info / Log

-file ext: .mjs
@hmmuaa hmmuaa changed the title _Search directory_ + _Search Content_ is excluding mjs files Search Directory + Search Content is excluding mjs files Jan 21, 2025
@harshad1
Copy link
Collaborator

I can't reproduce this .mjs files work fine for me

Is your file plain text?

@hmmuaa
Copy link
Author

hmmuaa commented Jan 21, 2025

Yes, it's just normal js file

Image

when I tried in-file find, I gotta

Image

n/when I tried search through dir, got nothing

Image

Image

@harshad1
Copy link
Collaborator

Interesting. Could you post this file here?

@hmmuaa
Copy link
Author

hmmuaa commented Jan 22, 2025

@hmmuaa
Copy link
Author

hmmuaa commented Jan 22, 2025

and my searching dir is under my work dir
dunno if is this necessary

Image

Image

@hmmuaa
Copy link
Author

hmmuaa commented Jan 22, 2025

Markor won't search any mjs/js files

I've tried several, such as finding 'import'
it may got results from many files, but indeed nothing found

I also tried rename .mjs to .js
still not work
--but if I rename to .md, I got it found

@hmmuaa hmmuaa changed the title Search Directory + Search Content is excluding mjs files Search Directory + Search Content is excluding mjs/js files Jan 22, 2025
@harshad1
Copy link
Collaborator

Can't reproduce.

I downloaded one of your files and it found it just fine. Even searched for non-ascii characters to check if that was the issue.

Image
Image

@hmmuaa
Copy link
Author

hmmuaa commented Jan 22, 2025

Well I also tried Marder
and it's new installed with initial config
and it's still not work

can't get an idea

@harshad1
Copy link
Collaborator

If you remove mjs from 'open with this app', restart markor, and try to open an mjs file, what happens?

@hmmuaa
Copy link
Author

hmmuaa commented Jan 22, 2025

Nothing wrong, just opened normally

@harshad1
Copy link
Collaborator

Then I have no idea :(

I can't reproduce this.

@hmmuaa
Copy link
Author

hmmuaa commented Jan 23, 2025

☺️
I'll upd if get further info

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants