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

Clicking on Search causes an Application Error #7199

Closed
1 task done
andrea-mauro opened this issue Mar 23, 2024 · 3 comments · Fixed by #7208
Closed
1 task done

Clicking on Search causes an Application Error #7199

andrea-mauro opened this issue Mar 23, 2024 · 3 comments · Fixed by #7208
Labels
B-bug Bug: general classification S-unverified Status: Unverified by maintainer

Comments

@andrea-mauro
Copy link
Contributor

Expected Behavior

When I click on the search input on the top left I expect the search input to open.

Actual Behavior

The following application error shows up

image

Reproduction Steps

  1. Click the Search input field on the top left of the screen
  2. An application Error shows up

Is there an existing issue for this?

Additional Information

Note that this is not happening when using the keyboard shortcut (⌘P on Mac).

Insomnia Version

develop branch

What operating system are you using?

macOS

Operating System Version

Mac OS Sonoma 14.3.1

Installation method

running the latest develop version

Last Known Working Insomnia version

No response

@andrea-mauro andrea-mauro added B-bug Bug: general classification S-unverified Status: Unverified by maintainer labels Mar 23, 2024
andrea-mauro added a commit to andrea-mauro/insomnia that referenced this issue Mar 23, 2024
@andrea-mauro
Copy link
Contributor Author

andrea-mauro commented Mar 23, 2024

I have a fix ready for this (see commit above). I'd be happy to publish a PR if this issue is confirmed.

@subnetmarco
Copy link
Member

@andrea-mauro if you have a fix, I suggest to open the PR so we can take a look at what the fix looks like. It works in my tests, but you may be addressing some edge case.

@andrea-mauro
Copy link
Contributor Author

Hi @subnetmarco . I've open the PR above

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B-bug Bug: general classification S-unverified Status: Unverified by maintainer
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants