-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Wmwragg/chat multi invite #2181
Conversation
… so it can user email as well mx userId
Can one of the admins verify this patch? |
Otherwise this is looking really good! Unsure whether to merge now and apply the cosmetic tweaks after the event or wait for this before cutting the freeze... |
Final question: is a deliberate design that the 'User ID, Name or email' prompt is always on the last line, rather than flowing inline and moving over the autocomplete bar to match? (superficially, removing the width of the textarea, setting display: inline-block and putting a relative offset on the autocomplete bar seems to work) |
(merged, although the stuff above still needs to be addressed somewhere) |
moving the feedback to #1882 so it's not lost. |
This should close #2131 and some of #1758 . A companion branch matrix-org/matrix-react-sdk:wmwragg/chat-multi-invite also needs to be merged.
Signed-off-by: William Wragg [email protected]