-
-
Notifications
You must be signed in to change notification settings - Fork 6.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
Indicate whether or not last message is yours #3012
Conversation
Try out |
Thanks! I think it worked. |
} else { | ||
return new SpannableString(getBody().getBody()); | ||
} | ||
return new SpannableString(getBody().getBody()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wrong indentation level.
I would favour a solution where I would also have the "delivered" information in the contact list. So additionally to the information that I send the last message, I would also know that it was delivered to the recipient. |
I agree, delivered would be helpful. What about some icons, maybe on the right side below the date? |
Looks like that would require a little more time to develop, and considering I don't have the icons... anyone is welcome to trash my PR and open another one :-) |
@volcano99 I could assist you with the icon's if thats helping you? |
My main problem is that I don't have a ton of experience coding, so I'm afraid I'd end up slowing things down :/ |
The normal one looks SO much better, why is it not being used in conversations too? |
Closing since this account was deleted. I'm in favor of displaying the delivery status of a message instead of a "You: " in the summary, since it provides even more information and still lets the user know it was a message sent from them. |
This should be a fix for #1363 (plus I fixed a typo)
Sorry if the PR looks like a mess, it's my first attempt to use github. Hope it's fine