Sitelet https://web.archive.org/web/20210903200609/https://github.com/laurent22/joplin/issues/5184
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

GoTo is not working at first try #5184

Closed
davidpoza opened this issue Jul 13, 2021 · 13 comments
Closed

GoTo is not working at first try #5184

davidpoza opened this issue Jul 13, 2021 · 13 comments

Comments

@davidpoza
Copy link

@davidpoza davidpoza commented Jul 13, 2021

GoTo command palette is not working as expected.

Environment

Joplin version: 2.2.1
Platform: linux
OS specifics: popOS 20.04

Steps to reproduce

  1. press CTRL+P to open command palette
  2. search any note
  3. press enter
  4. it does anything, not jumping to selected note
  5. repeat 1-4 steps again, and it works

Describe what you expected to happen

Previously it was working as expected (on 2.0.9)

@davidpoza davidpoza added the bug label Jul 13, 2021
@laurent22
Copy link
Owner

@laurent22 laurent22 commented Jul 13, 2021

Is there any error in the console? https://joplinapp.org/debugging

@davidpoza
Copy link
Author

@davidpoza davidpoza commented Jul 14, 2021 •

Sometimes it throws this error, but sometimes it doesn't

/tmp/.mount_JoplinZ6…GotoAnything.js:307 Uncaught (in promise) TypeError: Cannot read property 'makeItemIndexVisible' of null
    at Dialog.<anonymous> (/tmp/.mount_JoplinZ6…GotoAnything.js:307)
    at Generator.next (<anonymous>)
    at fulfilled (/tmp/.mount_JoplinZ6…s/GotoAnything.js:5)

@davidpoza
Copy link
Author

@davidpoza davidpoza commented Jul 24, 2021

Just to help out: I've tested 2.1.9 and this bug isn't present.

@laurent22
Copy link
Owner

@laurent22 laurent22 commented Aug 5, 2021

I'm not entirely sure what the issue is here, it might depend on your notes or some other factor. I've implemented a fix that may improve the situation, but if it happens again please let me know, and provide again the console statements.

@laurent22 laurent22 closed this in 5b7a5de Aug 5, 2021
@davidpoza
Copy link
Author

@davidpoza davidpoza commented Aug 11, 2021 •

Sorry, I need to open issue again.
I'm trying 2.2.6 prerelease but bug still exists.
It does nothing on first try, but second time it works.

I've made a screen capture:
Peek 2021-08-11 11-34

@pravsripad
Copy link

@pravsripad pravsripad commented Aug 13, 2021

I have the same issue (goto Cmd+P does not work first time). I do not see any error messages on the console.

Joplin for Desktop
Copyright © 2016-2021 Laurent Cozic
Joplin 2.2.7 (prod, darwin)
Client ID: 20e2a2bc858f4dce89aeffe2c7889ab0
Sync Version: 2
Profile Version: 39
Keychain Supported: Yes
Revision: 6d1a2e2

@gruen-f
Copy link

@gruen-f gruen-f commented Aug 16, 2021

Same here. Issue is not addressed.

@laurent22
Copy link
Owner

@laurent22 laurent22 commented Aug 17, 2021

I can't replicate it. Could someone try with all plugins disabled?

@pravsripad
Copy link

@pravsripad pravsripad commented Aug 17, 2021

I tried with all plugins disabled and I still face this issue. Sometimes see the below error on the console,

DevTools failed to load SourceMap: Could not load content for file:///private/var/folders/gw/r0dkp90n2mvcnkfbz8jqrnj80000gq/T/AppTranslocation/1F998594-1166-4ADC-9CCC-B845E87A798D/d/Joplin.app/Contents/Resources/app.asar/gui/note-viewer/pluginAssets/mermaid/mermaid.min.js.map: 
Joplin 2.3.3 (prod, darwin)
Client ID: 20e2a2bc858f4dce89aeffe2c7889ab0
Sync Version: 3
Profile Version: 39
Keychain Supported: Yes
Revision: f07636e

On a different profile with fewer number of notes, I cannot reproduce the issue even with the same plugins enabled.

@gruen-f
Copy link

@gruen-f gruen-f commented Aug 17, 2021

@3ter
Copy link

@3ter 3ter commented Aug 17, 2021 •

I replicated the issue on an empty collection. I then duplicated and imported notebooks to generate a larger number of notes and then the goto would only work after the second try after a time:
image
The first time it works and afterwards I need two tries.

@laurent22 laurent22 closed this in 4f5ad86 Aug 18, 2021
@banan314
Copy link

@banan314 banan314 commented Aug 21, 2021

It's resolved for me in version 2.4.1, Linux

@pravsripad
Copy link

@pravsripad pravsripad commented Aug 21, 2021

Works for me with 2.4.1 macOS as well. Thanks a lot for fixing this quickly.

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

Successfully merging a pull request may close this issue.

None yet
6 participants