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.

Sign up for GitHub

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

Jump to bottom

MessageBox is Not Responsive for Minimize mode #513

Open
Akshun-01 opened this issue Mar 12, 2024 · 5 comments · May be fixed by #514
Open

MessageBox is Not Responsive for Minimize mode #513

Akshun-01 opened this issue Mar 12, 2024 · 5 comments · May be fixed by #514
Labels
bug Something isn't working

Comments

@Akshun-01
Copy link
Contributor

The Message Box is not Responsive for Minimized mode of EC

When we use @ or / then the mention and command Modals take space and the message box disappears. This should not happen at least on the default screen sizes of EC (Minimized and Maximized mode)

Steps to reproduce:

  1. Use @ or / to bring up mention or command menu
  2. Observe the Message Box

Expected behavior:

Even on smaller screen Message Box should be visible.
image

Actual behavior:

The Message Box disappears when modal is triggered.
image
@Akshun-01 Akshun-01 added the bug Something isn't working label Mar 12, 2024
@Akshun-01
Copy link
Contributor Author

While fixing this issue, I saw another bug.
The mention menu is scrollable and arrow keys can be used to iterate through it but it does not shift, so that the currently selected item is in the modal window

currentMentionBug.mp4

@Akshun-01
Copy link
Contributor Author

Also unlike the mention menu, the command menu does not have the feature to scroll with the help of arrow keys

image

@Akshun-01 Akshun-01 linked a pull request Mar 12, 2024 that will close this issue
Draft
3 tasks
@Spiral-Memory
Copy link
Contributor

Great issue bro !

@Akshun-01
Copy link
Contributor Author

Thanks! ^^
Any feedback or changes? Would be great to have your insights.

@Spiral-Memory
Copy link
Contributor

Not right now. Complete your work, then I'll review it and let you know if we find anything interesting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

MessageBox Responsive Fix Akshun-01/EmbeddedChat
2 participants
@Spiral-Memory @Akshun-01