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

Bring up MemberInfo on avatar click in EventTile #225

Merged
merged 1 commit into from Mar 17, 2016

Conversation

Projects
None yet
2 participants
@aviraldg
Collaborator

aviraldg commented Mar 16, 2016

@ara4n

This comment has been minimized.

Member

ara4n commented Mar 17, 2016

this lgtm other than the rogue commented out code and the inlined CSS. i'll merge and fix there myself - thanks :)

ara4n added a commit that referenced this pull request Mar 17, 2016

Merge pull request #225 from aviraldg/fix-1174
Bring up MemberInfo on avatar click in EventTile

@ara4n ara4n merged commit e03d862 into matrix-org:develop Mar 17, 2016

ara4n added a commit that referenced this pull request Mar 17, 2016

@ara4n

This comment has been minimized.

Member

ara4n commented Mar 17, 2016

i tided this up a bit more in b617571 to match the onClick idiom used in the other components.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment