PrivateCoffee/element-web
4
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 4

Merge pull request #5037 from matrix-org/t3chguy/room-list/14705

Browse source 
Fix BaseAvatar wrongly using Buttons when it needs not
This commit is contained in:
Michael Telatynski 2020-07-23 09:27:20 +01:00 committed by GitHub
commit ff8f1af01e
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 5 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/components/views/avatars/BaseAvatar.tsx
View file

@ -134,7 +134,7 @@ const BaseAvatar = (props: IProps) => {
aria-hidden="true" /> aria-hidden="true" />
); );
if (onClick !== null) { if (onClick) {
return ( return (
<AccessibleButton <AccessibleButton
{...otherProps} {...otherProps}
@ -162,7 +162,7 @@ const BaseAvatar = (props: IProps) => {
} }
} }
if (onClick !== null) { if (onClick) {
return ( return (
<AccessibleButton <AccessibleButton
className={classNames("mx_BaseAvatar mx_BaseAvatar_image", className)} className={classNames("mx_BaseAvatar mx_BaseAvatar_image", className)}

5
test/components/views/messages/TextualBody-test.js
View file

@ -205,9 +205,8 @@ describe("<TextualBody />", () => {
expect(content.html()).toBe('<span class="mx_EventTile_body markdown-body" dir="auto">' + expect(content.html()).toBe('<span class="mx_EventTile_body markdown-body" dir="auto">' +
'Hey <span>' + 'Hey <span>' +
'<a class="mx_Pill mx_UserPill" title="@user:server">' + '<a class="mx_Pill mx_UserPill" title="@user:server">' +
'<img src="mxc://avatar.url/image.png" style="width: 16px; height: 16px;" ' + '<img class="mx_BaseAvatar mx_BaseAvatar_image" src="mxc://avatar.url/image.png" ' +
'title="@member:domain.bla" alt="" aria-hidden="true" role="button" tabindex="0" ' + 'style="width: 16px; height: 16px;" title="@member:domain.bla" alt="" aria-hidden="true">Member</a>' +
'class="mx_AccessibleButton mx_BaseAvatar mx_BaseAvatar_image">Member</a>' +
'</span></span>'); '</span></span>');
}); });
}); });