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

Erratic Emoji Picker Behavior #1249

Open
canny bot opened this issue Apr 16, 2024 · 6 comments
Open

Erratic Emoji Picker Behavior #1249

canny bot opened this issue Apr 16, 2024 · 6 comments
Labels
bug Something isn't working maint-pe Issues for Maint releases / Product Engine team:viewer
Milestone

Comments

@canny
Copy link

canny bot commented Apr 16, 2024

Open emoji picker, choose more, start to search for an emoji, don't choose one, click the X. Focus is not returned to the chat box, and remains on the now closed search box.

https://secondlife.canny.io/admin/board/bug-reports/p/erratic-emoji-picker-behavior

UPDATE:

Since the Emoji Picker is focusless, and there is no any "search box", the more correct bug description could be as follows:

When EmojiPicker is opened by "More" button and closed by "X" button, the text input field stops reacting on keyboard

Steps to reprodice:

  • Open the "CONVERSATIONS" floater
  • Click smile button right of the text input field
  • Click "More" button below the smile button
  • Close "CHOOSE EMOJI" floater by [X] button

Actual behavior:
Text input field doesn't accept text input and caret positioning
(user presses any symbol keys, arrow keys, backspace key with no effect)

Expected behavior:
Text input field accepts text input and caret positioning

Copy link
Author

canny bot commented Apr 16, 2024

This issue has been linked to a Canny post: Erratic Emoji Picker Behavior 🎉

@AtlasLinden AtlasLinden added bug Something isn't working team:viewer maint-pe Issues for Maint releases / Product Engine labels Apr 16, 2024
@AtlasLinden AtlasLinden added this to the Maintenance B milestone Apr 16, 2024
@LLGuru LLGuru self-assigned this Apr 17, 2024
LLGuru added a commit that referenced this issue Apr 18, 2024
LLGuru added a commit that referenced this issue Apr 18, 2024
LLGuru added a commit that referenced this issue Apr 18, 2024
LLGuru added a commit that referenced this issue Apr 18, 2024
LLGuru added a commit that referenced this issue Apr 19, 2024
@LLGuru
Copy link
Contributor

LLGuru commented Apr 19, 2024

Fixed by #1276

@LLGuru LLGuru removed their assignment Apr 19, 2024
@katerynamproductengine
Copy link

Failed QA.

Text input field still doesn't accept text input and caret positioning.
This bug is constantly reproduced when using Latin keyboard and occasionally reproduced when using Cyrillic keyboard.

Verified on the Second Life Release 7.1.7.8820696922 (64bit) on Win10/OSX in the scope of secondlife/iqa#209.

@marchcat
Copy link
Contributor

marchcat commented May 10, 2024

The fix #1276 was merged into Maint X. Please recheck there.

@AtlasLinden
Copy link

Maint X is being promoted to default viewer
Setting this back to Maint B to be tested after merge

@sergiiiproductengine
Copy link

Passed QA.

Verified on the Second Life Release 7.1.8.9130881608 (64bit) on Win10/OSX in the scope of https://github.com/secondlife/iqa/issues/237.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working maint-pe Issues for Maint releases / Product Engine team:viewer
Projects
None yet
Development

No branches or pull requests

5 participants