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

Typing two keys quickly in spotlight dialog closes the dialog #28493

Closed
david-crespo opened this issue Nov 19, 2024 · 8 comments
Closed

Typing two keys quickly in spotlight dialog closes the dialog #28493

david-crespo opened this issue Nov 19, 2024 · 8 comments
Labels
A-Spotlight-Search The new search dialog available in develop and Nightly O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect

Comments

@david-crespo
Copy link

david-crespo commented Nov 19, 2024

Steps to reproduce

After upgrading to 1.11.86:

  1. Press cmd-k to open room quick search
  2. Type any two keys quickly

I was able to confirm that downgrading to 1.11.85 fixes the issue.

Outcome

What did you expect?

I expect the search dialog to stay open and the input field shows the two keys I pressed.

What happened instead?

The dialog closes, and the message input is focused instead. This means that if I type, for example, three keys and press enter, the first two go into the room quick search and the third + enter go into the message box. I have been sending a lot of one-letter messages today.

2024-11-19-element-room-search-bug.mp4

Operating system

macOS

Application version

1.11.86

How did you install the app?

https://element.io/app-for-productivity, but that was a long time ago. This was just an in-app upgrade.

Homeserver

https://matrix.oxide.computer

Will you send logs?

Yes

@dosubot dosubot bot added A-Spotlight-Search The new search dialog available in develop and Nightly O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Minor Impairs non-critical functionality or suitable workarounds exist labels Nov 19, 2024
@david-crespo david-crespo changed the title Typing two keys quickly in the room quick search closes the dialog Typing two keys quickly in spotlight dialog closes the dialog Nov 19, 2024
@david-crespo
Copy link
Author

david-crespo commented Nov 19, 2024

I'm only a simple country web developer, but #28452 from last week changes the node that gets focused on various events in the spotlight dialog.

@t3chguy
Copy link
Member

t3chguy commented Nov 19, 2024

I'm only a simple country web developer, but #28452 from last week changes the node that gets focused on various events in the spotlight dialog.

This PR is not included in 1.11.86 as it was merged after the RC.

@MidhunSureshR you were handling fixing this and believed it to be due to React 18 double-rendering which only affects development builds, this issue claims otherwise, can you confirm?

@MidhunSureshR MidhunSureshR added S-Major Severely degrades major functionality or product features, with no satisfactory workaround O-Occasional Affects or can be seen by some users regularly or most users rarely O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience and removed S-Minor Impairs non-critical functionality or suitable workarounds exist O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience O-Occasional Affects or can be seen by some users regularly or most users rarely labels Nov 20, 2024
@MidhunSureshR
Copy link
Member

I can repro on 1.11.86 but not on 1.11.85.

@MidhunSureshR
Copy link
Member

From a git bisect, this is from #28336

@Ajmalajjuca
Copy link

move to 1.11.85.

@t3chguy
Copy link
Member

t3chguy commented Nov 20, 2024

This is already fixed on develop and will be in the next release

@andrew-demb
Copy link

@t3chguy can you please tell us, approximately when the next release will be published?

@t3chguy
Copy link
Member

t3chguy commented Nov 26, 2024

RC is today, Release next Tuesday

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Spotlight-Search The new search dialog available in develop and Nightly O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect
Projects
None yet
Development

No branches or pull requests

5 participants