diff --git a/packages/ui/src/components/Header/Header.module.css b/packages/ui/src/components/Header/Header.module.css index e6ea55e363..2e3ca28d3b 100644 --- a/packages/ui/src/components/Header/Header.module.css +++ b/packages/ui/src/components/Header/Header.module.css @@ -54,9 +54,19 @@ align-items: center; gap: var(--bui-space-2); flex-wrap: wrap; + list-style: none; + margin: 0; + padding: 0; } - .bui-HeaderTagDivider { + .bui-HeaderTag { + display: flex; + align-items: center; + gap: var(--bui-space-2); + } + + .bui-HeaderTag + .bui-HeaderTag::before { + content: ''; width: 3px; height: 3px; border-radius: 50%; @@ -70,6 +80,7 @@ align-items: center; gap: var(--bui-space-5); flex-wrap: wrap; + margin: 0; } .bui-HeaderMetaItem { @@ -77,5 +88,9 @@ flex-direction: row; align-items: center; gap: var(--bui-space-2); + + dd { + margin: 0; + } } } diff --git a/packages/ui/src/components/Header/Header.tsx b/packages/ui/src/components/Header/Header.tsx index 0098fa00d4..5d41d6ca0f 100644 --- a/packages/ui/src/components/Header/Header.tsx +++ b/packages/ui/src/components/Header/Header.tsx @@ -23,7 +23,7 @@ import { HeaderDefinition } from './definition'; import { Container } from '../Container'; import { Lexer } from 'marked'; import { Link } from '../Link'; -import { Fragment } from 'react/jsx-runtime'; +import { Fragment } from 'react'; // Reject javascript:/vbscript:/data: URIs to prevent XSS via description links. const UNSAFE_HREF_RE = /^(javascript:|vbscript:|data:)/i; @@ -78,10 +78,12 @@ export const Header = (props: HeaderProps) => { return ( {tags && tags.length > 0 && ( -
+
+ )}
@@ -135,20 +137,24 @@ export const Header = (props: HeaderProps) => { )} {metadata && metadata.length > 0 && ( -
+
{metadata.map((item, i) => (
- - {item.label} - - {typeof item.value === 'string' ? ( - {item.value} - ) : ( - item.value - )} +
+ + {item.label} + +
+
+ {typeof item.value === 'string' ? ( + {item.value} + ) : ( + item.value + )} +
))} -
+ )} {tabs && (
diff --git a/packages/ui/src/components/Header/HeaderMetadataStatus.tsx b/packages/ui/src/components/Header/HeaderMetadataStatus.tsx index 75c9ea778f..2bf2effa8f 100644 --- a/packages/ui/src/components/Header/HeaderMetadataStatus.tsx +++ b/packages/ui/src/components/Header/HeaderMetadataStatus.tsx @@ -32,7 +32,11 @@ export const HeaderMetadataStatus = ({ }: HeaderMetadataStatusItem) => { return (
- + {href ? ( diff --git a/packages/ui/src/components/Header/HeaderMetadataUsers.module.css b/packages/ui/src/components/Header/HeaderMetadataUsers.module.css index 745e768d56..0b84734b81 100644 --- a/packages/ui/src/components/Header/HeaderMetadataUsers.module.css +++ b/packages/ui/src/components/Header/HeaderMetadataUsers.module.css @@ -29,6 +29,9 @@ flex-direction: row; align-items: center; gap: var(--bui-space-1); + list-style: none; + margin: 0; + padding: 0; } .avatarLink { diff --git a/packages/ui/src/components/Header/HeaderMetadataUsers.tsx b/packages/ui/src/components/Header/HeaderMetadataUsers.tsx index e644811899..ce1688643f 100644 --- a/packages/ui/src/components/Header/HeaderMetadataUsers.tsx +++ b/packages/ui/src/components/Header/HeaderMetadataUsers.tsx @@ -39,43 +39,10 @@ export const HeaderMetadataUsers = ({ if (users.length === 1) { const user = users[0]; - const avatar = ( - - ); return (
{user.href ? ( - - {avatar} - - ) : ( - avatar - )} - {user.href ? ( - - {user.name} - - ) : ( - {user.name} - )} -
- ); - } - - return ( -
- {users.map(user => ( - - {user.href ? ( + <> - ) : ( - - - - )} - {user.name} - + + {user.name} + + + ) : ( + <> + + {user.name} + + )} +
+ ); + } + + return ( +
    + {users.map(user => ( +
  • + + {user.href ? ( + + + + ) : ( + + + + )} + {user.name} + +
  • ))} -
+ ); }; diff --git a/packages/ui/src/components/Header/definition.ts b/packages/ui/src/components/Header/definition.ts index 7c9878d7e9..c0420dade5 100644 --- a/packages/ui/src/components/Header/definition.ts +++ b/packages/ui/src/components/Header/definition.ts @@ -31,7 +31,7 @@ export const HeaderDefinition = defineComponent()({ tabsWrapper: 'bui-HeaderTabsWrapper', controls: 'bui-HeaderControls', tags: 'bui-HeaderTags', - tagDivider: 'bui-HeaderTagDivider', + tag: 'bui-HeaderTag', description: 'bui-HeaderDescription', metaRow: 'bui-HeaderMetaRow', metaItem: 'bui-HeaderMetaItem',