-
-
Notifications
You must be signed in to change notification settings - Fork 379
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 checkmark to verified links and better constraints to account fields #922
Conversation
connyduck
commented
Nov 18, 2018
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.
LGTM!
LinkHelper.setClickableText(viewHolder.valueTextView, emojifiedValue, null, linkListener) | ||
|
||
if(field.verifiedAt != null) { | ||
TextViewCompat.setCompoundDrawablesRelativeWithIntrinsicBounds(viewHolder.valueTextView, 0, 0, R.drawable.ic_check_circle, 0) |
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.
Do we still need to use compat?
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.
No you are right, we don't. I will remove it from everywhere else as well.
app:layout_constrainedWidth="true" | ||
app:layout_constraintWidth_max="160dp" | ||
tools:text="Field title " /> | ||
app:layout_constraintWidth_percent=".3" |
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.
Not clear why .3 is needed here. Perhaps a comment is needed.
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.
I just think it looks best, no technical reason