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

Desktop: Goto Anything modal closes even when you click inside #3028

Closed
genneko opened this issue Apr 12, 2020 · 1 comment · Fixed by #3029
Closed

Desktop: Goto Anything modal closes even when you click inside #3028

genneko opened this issue Apr 12, 2020 · 1 comment · Fixed by #3029
Labels
bug It's a bug desktop All desktop platforms high High priority issues

Comments

@genneko
Copy link
Contributor

genneko commented Apr 12, 2020

"Goto Anything" modal dialog closes even when you click on it (including the text field and the "?" icon).
This prevents the help text for the dialog from showing up when you click "?".
I don't think it's a big issue but it would be better if it closes only when you click outside the modal.

Environment

Joplin version: Electron v1.0.199
Platform: Linux, Windows
OS specifics: Ubuntu 18.04, Windows 10

Steps to reproduce

  1. Start Joplin Desktop app.
  2. Select "Tools" > "Goto Anything" or hit Ctrl-G to open "Goto Anything" modal.
  3. Click anywhere on the modal (the search text field, "?" icon or margin area). You will see the modal closes on click.

Describe what you expected to happen

  • Clicking "?" toggles the help text above the text field.
  • Clicking on the modal doesn't close it.
@genneko genneko added the bug It's a bug label Apr 12, 2020
@anjulalk
Copy link
Contributor

I think this is caused by #2940. Most probably it incorrectly binds the onClick event to the entire page, closing the Go to anything modal wherever you click.

I'll look into this.

@laurent22 laurent22 added high High priority issues desktop All desktop platforms labels Apr 12, 2020
laurent22 added a commit that referenced this issue Apr 12, 2020
… the modal (#3029)

* Fix Go To Anything closing when clicking inside the modal

* Update GotoAnything.jsx

* Update GotoAnything.jsx

Co-authored-by: Laurent Cozic <laurent22@users.noreply.github.com>
@lock lock bot locked and limited conversation to collaborators Apr 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug It's a bug desktop All desktop platforms high High priority issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants