-
Notifications
You must be signed in to change notification settings - Fork 50
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
Add "Send message" to People cards #257
Comments
@orangiene Can you please take a look here? Do we have a concept regarding the button colors/types for cards? |
I like the idea, let's do it. |
Thanks @orangiene . As the buttons' type list is https://github.com/humhub/humhub/blob/master/protected/humhub/widgets/BootstrapComponent.php#L47, can I use |
@luke- #258 but because of this problem: humhub/humhub#5622 |
@luke- We have an unresolved discussion here: #258 (comment) |
@luke- @marc-farre I think it's a great opportunity to use an icon button that could turn into a floating button in the future. What do you think about it? |
@Eladnarlea Thanks! Yes, I think it's a very good idea. |
@luke- a menu like this is less intuitive than the icon button. Also for mobile it is less practical. |
@Eladnarlea I see your point, but for me the people cards are a bit bloated. If we had a "hidden" decent menu, we could also put other additional not that important options (Block, Edit as Admin, ...) in there. Or without a new menu: We could always display the mail symbol at the bottom (and either +Friends or +Follow - depending on the HumHub configuration. It's confusing for many users anyway, the difference between follow/friends) For me, however, the solution with the "message" button at the top would also be fine. Even if it's not my favorite. |
I would advise against applying too many options to each page of the software. We should first define what the "people cards" are for and what you can do on the page. Blocking, for example, is a fairly specific action that is rarely used in mass processing. So I don't think it's necessarily relevant to the People Cards page.
I like this, I think it's also confusing and we should offer either/or. "Follow" is also an action that should be configured when on the specific profile.
@luke- let me think of another way to deal with the "bloated" cards, I would like to provide a different approach. |
What do you think about adding a "Send message" button to the cards in the People page?
If you agree with this idea, I can send a PR adding an event to add an item on
PeopleActionButtons
widget.The text was updated successfully, but these errors were encountered: