Skip to content

Commit

Permalink
[Workplace Search] Remove user icon from source details (#88218)
Browse files Browse the repository at this point in the history
* [Workplace Search] Remove user icon from source details

Removes user icons from source details group list, since the users don't exist in Kibana.

* Refactor to remove entire GroupAvatars component

* Remove unused interface

* Remove flex group

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
  • Loading branch information
scottybollinger and kibanamachine committed Jan 15, 2021
1 parent ca42e9a commit 86218a4
Showing 1 changed file with 3 additions and 37 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ import React from 'react';
import { useValues } from 'kea';

import {
EuiAvatar,
EuiEmptyPrompt,
EuiFlexGroup,
EuiFlexItem,
Expand Down Expand Up @@ -40,7 +39,6 @@ import {
} from '../../../routes';

import { AppLogic } from '../../../app_logic';
import { User } from '../../../types';

import { ComponentLoader } from '../../../components/shared/component_loader';
import { CredentialItem } from '../../../components/shared/credential_item';
Expand Down Expand Up @@ -223,31 +221,6 @@ export const Overview: React.FC = () => {
};

const GroupsSummary = () => {
const GroupAvatars = ({ users }: { users: User[] }) => {
const MAX_USERS = 4;
return (
<EuiFlexGroup gutterSize="xs" alignItems="center">
{users.slice(0, MAX_USERS).map((user) => (
<EuiFlexItem key={user.id}>
<EuiAvatar
size="s"
initials={user.initials}
name={user.name || user.initials}
imageUrl={user.pictureUrl || ''}
/>
</EuiFlexItem>
))}
{users.slice(MAX_USERS).length > 0 && (
<EuiFlexItem>
<EuiText color="subdued" size="xs">
<strong>+{users.slice(MAX_USERS).length}</strong>
</EuiText>
</EuiFlexItem>
)}
</EuiFlexGroup>
);
};

return !groups.length ? null : (
<>
<EuiText>
Expand All @@ -262,16 +235,9 @@ export const Overview: React.FC = () => {
data-test-subj="SourceGroupLink"
className="euiPanel--inset"
>
<EuiFlexGroup alignItems="center">
<EuiFlexItem>
<EuiText size="s" className="eui-textTruncate">
<strong>{group.name}</strong>
</EuiText>
</EuiFlexItem>
<EuiFlexItem grow={false}>
<GroupAvatars users={group.users} />
</EuiFlexItem>
</EuiFlexGroup>
<EuiText size="s" className="eui-textTruncate">
<strong>{group.name}</strong>
</EuiText>
</EuiPanelTo>
</EuiFlexItem>
))}
Expand Down

0 comments on commit 86218a4

Please sign in to comment.