diff --git a/openmetadata-ui/src/main/resources/ui/src/components/ManageTab/ManageTab.component.tsx b/openmetadata-ui/src/main/resources/ui/src/components/ManageTab/ManageTab.component.tsx index ce95a9ac022..ee2b6123946 100644 --- a/openmetadata-ui/src/main/resources/ui/src/components/ManageTab/ManageTab.component.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/components/ManageTab/ManageTab.component.tsx @@ -12,6 +12,7 @@ */ import { AxiosResponse } from 'axios'; +import classNames from 'classnames'; import { isEmpty } from 'lodash'; import { observer } from 'mobx-react'; import { TableDetail } from 'Models'; @@ -181,6 +182,8 @@ const ManageTab: FunctionComponent = ({ }); }; + const ownerName = getOwnerById(); + useEffect(() => { getTierData(); }, []); @@ -225,7 +228,17 @@ const ManageTab: FunctionComponent = ({ theme="primary" variant="link" onClick={() => setListVisible((visible) => !visible)}> - {getOwnerById() || 'Select Owner'} + {ownerName ? ( + 32, + })} + title={ownerName}> + {ownerName} + + ) : ( + 'Select Owner' + )} { + return; + }} /> -

{item.name}

+

+ {item.name} +

) : ( diff --git a/openmetadata-ui/src/main/resources/ui/src/components/dropdown/DropDown.tsx b/openmetadata-ui/src/main/resources/ui/src/components/dropdown/DropDown.tsx index af0c3989790..6160d1851df 100644 --- a/openmetadata-ui/src/main/resources/ui/src/components/dropdown/DropDown.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/components/dropdown/DropDown.tsx @@ -85,9 +85,10 @@ const DropDown: React.FC = ({ if (selectedItems?.includes(item.value as string)) { return (

+ style={{ margin: '2px' }} + title={item.name as string}> {item.name}

); diff --git a/openmetadata-ui/src/main/resources/ui/src/components/dropdown/DropDownList.tsx b/openmetadata-ui/src/main/resources/ui/src/components/dropdown/DropDownList.tsx index 85c6645017b..44cda04121a 100644 --- a/openmetadata-ui/src/main/resources/ui/src/components/dropdown/DropDownList.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/components/dropdown/DropDownList.tsx @@ -69,7 +69,7 @@ const DropDownList: FunctionComponent = ({ const getDropDownElement = (item: DropDownListItem, index: number) => { return ( - = ({ key={index} role="menuitem" onClick={(e) => onSelect && onSelect(e, item.value)}> - {item.name} - +

+ {item.name} +

+ ); }; diff --git a/openmetadata-ui/src/main/resources/ui/src/pages/teams/index.tsx b/openmetadata-ui/src/main/resources/ui/src/pages/teams/index.tsx index 5790f039735..72dedc48e8d 100644 --- a/openmetadata-ui/src/main/resources/ui/src/pages/teams/index.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/pages/teams/index.tsx @@ -120,9 +120,13 @@ const TeamsPage = () => { ) ) { errData['name'] = 'Name already exists'; + } else if (data.name.length < 1 || data.name.length > 128) { + errData['name'] = 'Name size must be between 1 and 128'; } if (!data.displayName?.trim()) { errData['displayName'] = 'Display name is required'; + } else if (data.displayName.length < 1 || data.displayName.length > 128) { + errData['displayName'] = 'Display name size must be between 1 and 128'; } setErrorData(errData); @@ -424,7 +428,9 @@ const TeamsPage = () => {
-
+
{currentTeam?.displayName}
{ rel="noopener noreferrer" target={data.openInNewTab ? '_blank' : '_self'}> <> - {displayVal} + 32, + })} + title={displayVal as string}> + {displayVal} + {data.openInNewTab && ( { ) : ( - displayVal + <> + {data.key === 'Owner' ? ( + 32 } + )} + title={displayVal as string}> + {displayVal} + + ) : ( + {displayVal} + )} + )} ) : null} diff --git a/openmetadata-ui/src/main/resources/ui/src/utils/EntityVersionUtils.tsx b/openmetadata-ui/src/main/resources/ui/src/utils/EntityVersionUtils.tsx index 883e76c42a2..61b7b342292 100644 --- a/openmetadata-ui/src/main/resources/ui/src/utils/EntityVersionUtils.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/utils/EntityVersionUtils.tsx @@ -356,6 +356,9 @@ export const feedSummaryFromatter = ( } case fieldChange?.name === 'owner': { + const ownerName = + getOwnerName(newValue?.id as string) || + getOwnerName(value?.id as string); const ownerText = !isEmpty(oldValue) && !isEmpty(newValue) ? ( @@ -363,11 +366,11 @@ export const feedSummaryFromatter = ( - {getOwnerName(newValue?.id as string)} + {ownerName} ) : ( - - {getOwnerName(newValue?.id as string)} + + {ownerName} )} @@ -377,17 +380,21 @@ export const feedSummaryFromatter = ( - {getOwnerName(value?.id as string)} + {ownerName} ) : ( - - {getOwnerName(value?.id as string)} + + {ownerName} )} ); summary = ( -

+

32, + })} + key={uniqueId()}> {`Assigned ownership to`} {ownerText}