-
Notifications
You must be signed in to change notification settings - Fork 10.7k
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
Added dropdown for message options #829
Conversation
Hold on this. May close it out. @rodrigok has something he was trying out that looked much cooler. :) |
@geekgonecrazy done :) |
nice! 👍 |
Is that on hover or on click? Looks like on click, wonder how it would work with on hover.. |
I think you should not be able to open two dropdowns.. I mean, if you click outside the opened dropdown, it should close. what you guys mean? |
👍 |
Would that get tricky to actually click on one of them? We can try it
I agree. Clicking outside should close it, and it can get messy with multiple up. |
Yes I agree, there should be a max of only one opened up at a time. I'm wondering if it should show up on hovering instead of clicking on the cog? That way it's one less click? |
@graywolf336, can be on hover, but hover don't work for mobile. |
@rodrigok I would think mobile you would tap and hold the message to bring up a list of actions on the message, like the report abuse way is. |
See f60eadc |
@graywolf336 I'm thinking something more like this on tap: |
@rodrigok - This really looks cool. 👍 |
@rodrigok Oh yeah that'll work too 👍 |
Change log level of gettranscript from debug to error
Behavior could probably use some tweaking. Thoughts @RocketChat/owners?
Related to: #828 and #310