@scm-manager/ui-components 3.10.2-20250728-184027 → 3.10.2-20250802-135601

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@scm-manager/ui-components",
-  "version": "3.10.2-20250728-184027",
+  "version": "3.10.2-20250802-135601",
   "description": "UI Components for SCM-Manager and its plugins",
   "main": "src/index.ts",
   "files": [
@@ -32,8 +32,8 @@
     "react-query": "^3.39.2"
   },
   "devDependencies": {
-    "@scm-manager/ui-tests": "3.10.2-20250728-184027",
-    "@scm-manager/ui-types": "3.10.2-20250728-184027",
+    "@scm-manager/ui-tests": "3.10.2-20250802-135601",
+    "@scm-manager/ui-types": "3.10.2-20250802-135601",
     "@types/fetch-mock": "^7.3.1",
     "@types/react-select": "^2.0.19",
     "@types/unist": "^2.0.3",
@@ -68,17 +68,17 @@
     "@scm-manager/jest-preset": "^2.14.1",
     "@scm-manager/prettier-config": "^2.12.0",
     "@scm-manager/tsconfig": "^2.13.0",
-    "@scm-manager/ui-syntaxhighlighting": "3.10.2-20250728-184027",
-    "@scm-manager/ui-shortcuts": "3.10.2-20250728-184027",
-    "@scm-manager/ui-text": "3.10.2-20250728-184027"
+    "@scm-manager/ui-syntaxhighlighting": "3.10.2-20250802-135601",
+    "@scm-manager/ui-shortcuts": "3.10.2-20250802-135601",
+    "@scm-manager/ui-text": "3.10.2-20250802-135601"
   },
   "dependencies": {
-    "@scm-manager/ui-core": "3.10.2-20250728-184027",
-    "@scm-manager/ui-overlays": "3.10.2-20250728-184027",
-    "@scm-manager/ui-layout": "3.10.2-20250728-184027",
-    "@scm-manager/ui-buttons": "3.10.2-20250728-184027",
-    "@scm-manager/ui-api": "3.10.2-20250728-184027",
-    "@scm-manager/ui-extensions": "3.10.2-20250728-184027",
+    "@scm-manager/ui-core": "3.10.2-20250802-135601",
+    "@scm-manager/ui-overlays": "3.10.2-20250802-135601",
+    "@scm-manager/ui-layout": "3.10.2-20250802-135601",
+    "@scm-manager/ui-buttons": "3.10.2-20250802-135601",
+    "@scm-manager/ui-api": "3.10.2-20250802-135601",
+    "@scm-manager/ui-extensions": "3.10.2-20250802-135601",
     "deepmerge": "^4.2.2",
     "hast-util-sanitize": "^3.0.2",
     "react-diff-view": "^2.4.10",

package/src/__snapshots__/storyshots.test.ts.snap

@@ -20389,7 +20389,7 @@ exports[`Storyshots Repositories/Changesets Co-Authors with avatar 1`] = `
                 className="media-left mt-2 mr-2"
               >
                 <div
-                  className="SingleChangeset__FixedSizedAvatar-sc-ytpqp9-0 dLAXC image"
+                  className="SingleChangeset__FixedSizedAvatar-sc-ytpqp9-0 gwbxYG image"
                 >
                   <img
                     alt="SCM Administrator"
@@ -20448,6 +20448,7 @@ exports[`Storyshots Repositories/Changesets Co-Authors with avatar 1`] = `
                         <img
                           alt="Ford Prefect"
                           className="ContributorAvatar-sc-1yz8zn-0 lhgGHS"
+                          crossOrigin="anonymous"
                           src="https://robohash.org/ford.prefect@hitchhiker.com"
                         />
                       </a>
@@ -20458,6 +20459,7 @@ exports[`Storyshots Repositories/Changesets Co-Authors with avatar 1`] = `
                         <img
                           alt="Zaphod Beeblebrox"
                           className="ContributorAvatar-sc-1yz8zn-0 lhgGHS"
+                          crossOrigin="anonymous"
                           src="https://robohash.org/zaphod.beeblebrox@hitchhiker.cm"
                         />
                       </a>
@@ -20468,6 +20470,7 @@ exports[`Storyshots Repositories/Changesets Co-Authors with avatar 1`] = `
                         <img
                           alt="Tricia Marie McMillan"
                           className="ContributorAvatar-sc-1yz8zn-0 lhgGHS"
+                          crossOrigin="anonymous"
                           src="https://robohash.org/trillian@hitchhiker.cm"
                         />
                       </a>
@@ -20564,7 +20567,7 @@ exports[`Storyshots Repositories/Changesets Commiter and Co-Authors with avatar
                 className="media-left mt-2 mr-2"
               >
                 <div
-                  className="SingleChangeset__FixedSizedAvatar-sc-ytpqp9-0 dLAXC image"
+                  className="SingleChangeset__FixedSizedAvatar-sc-ytpqp9-0 gwbxYG image"
                 >
                   <img
                     alt="SCM Administrator"
@@ -20618,6 +20621,7 @@ exports[`Storyshots Repositories/Changesets Commiter and Co-Authors with avatar
                       <img
                         alt="Zaphod Beeblebrox"
                         className="ContributorAvatar-sc-1yz8zn-0 lhgGHS"
+                        crossOrigin="anonymous"
                         src="https://robohash.org/zaphod.beeblebrox@hitchhiker.cm"
                       />
                     </a>
@@ -20633,6 +20637,7 @@ exports[`Storyshots Repositories/Changesets Commiter and Co-Authors with avatar
                       <img
                         alt="Ford Prefect"
                         className="ContributorAvatar-sc-1yz8zn-0 lhgGHS"
+                        crossOrigin="anonymous"
                         src="https://robohash.org/ford.prefect@hitchhiker.com"
                       />
                     </a>
@@ -21637,7 +21642,7 @@ exports[`Storyshots Repositories/Changesets With avatar 1`] = `
                 className="media-left mt-2 mr-2"
               >
                 <div
-                  className="SingleChangeset__FixedSizedAvatar-sc-ytpqp9-0 dLAXC image"
+                  className="SingleChangeset__FixedSizedAvatar-sc-ytpqp9-0 gwbxYG image"
                 >
                   <img
                     alt="SCM Administrator"

package/src/repos/DiffTypes.ts

@@ -14,9 +14,9 @@
  * along with this program. If not, see https://www.gnu.org/licenses/.
  */
 
-import { ReactNode } from "react";
+import { FC, ReactNode } from "react";
 import { DefaultCollapsed } from "./defaultCollapsed";
-import { Change, Hunk, FileDiff as File } from "@scm-manager/ui-types";
+import { Change, Hunk, FileDiff as File, FileChangeType } from "@scm-manager/ui-types";
 
 export type ChangeEvent = {
   change: Change;
@@ -31,6 +31,18 @@ export type AnnotationFactoryContext = BaseContext;
 
 export type FileAnnotationFactory = (file: File) => ReactNode[];
 
+export type FileTreeNodeWrapper = FC<{
+  name: string;
+  path: string;
+  changeType?: FileChangeType;
+  iconName: string;
+  iconColor: string;
+  isFile: boolean;
+  originalIcon: ReactNode;
+  originalLabel: ReactNode;
+  isCurrentFile: boolean;
+}>;
+
 // key = change id, value = react component
 export type AnnotationFactory = (context: AnnotationFactoryContext) => {
   [key: string]: any;

package/src/repos/changesets/ChangesetAuthor.tsx

@@ -57,11 +57,11 @@ const ContributorWithAvatar: FC<PersonAvatarProps> = ({ person, avatar }) => {
   if (person.mail) {
     return (
       <a href={"mailto:" + person.mail} title={t("changeset.contributors.mailto") + " " + person.mail}>
-        <ContributorAvatar src={avatar} alt={person.name} />
+        <ContributorAvatar src={avatar} alt={person.name} crossOrigin="anonymous" />
       </a>
     );
   }
-  return <ContributorAvatar src={avatar} alt={person.name} title={person.name} />;
+  return <ContributorAvatar src={avatar} alt={person.name} title={person.name} crossOrigin="anonymous" />;
 };
 
 export const SingleContributor: FC<PersonProps> = ({ person, className, displayTextOnly }) => {

package/src/repos/changesets/SingleChangeset.tsx

@@ -37,7 +37,7 @@ type Props = {
 const FixedSizedAvatar = styled.div`
   width: 35px;
   height: 35px;
-  min-width: fit-content;
+  overflow: hidden;
 `;
 
 const FullWidthDiv = styled.div`

package/src/repos/diff/DiffFileTree.tsx

@@ -18,12 +18,19 @@ import React, { FC } from "react";
 import { Link } from "react-router-dom";
 import { useTranslation } from "react-i18next";
 import classNames from "classnames";
-import { FileTree } from "@scm-manager/ui-types";
+import { FileChangeType, FileTree } from "@scm-manager/ui-types";
 import { FileDiffContent, StackedSpan, StyledIcon } from "./styledElements";
+import { FileTreeNodeWrapper } from "../DiffTypes";
 
-type Props = { tree: FileTree; currentFile: string; setCurrentFile: (path: string) => void; gap?: number };
+type Props = {
+  tree: FileTree;
+  currentFile: string;
+  setCurrentFile: (path: string) => void;
+  gap?: number;
+  FileTreeNodeWrapper?: FileTreeNodeWrapper;
+};
 
-const DiffFileTree: FC<Props> = ({ tree, currentFile, setCurrentFile, gap = 15 }) => {
+const DiffFileTree: FC<Props> = ({ tree, currentFile, setCurrentFile, gap = 15, FileTreeNodeWrapper }) => {
   return (
     <FileDiffContent gap={gap}>
       {Object.keys(tree.children).map((key) => (
@@ -33,6 +40,7 @@ const DiffFileTree: FC<Props> = ({ tree, currentFile, setCurrentFile, gap = 15 }
           parentPath=""
           currentFile={currentFile}
           setCurrentFile={setCurrentFile}
+          FileTreeNodeWrapper={FileTreeNodeWrapper}
         />
       ))}
     </FileDiffContent>
@@ -41,9 +49,13 @@ const DiffFileTree: FC<Props> = ({ tree, currentFile, setCurrentFile, gap = 15 }
 
 export default DiffFileTree;
 
-type ChangeType = "add" | "modify" | "delete" | "rename" | "copy";
-
-type NodeProps = { node: FileTree; parentPath: string; currentFile: string; setCurrentFile: (path: string) => void };
+type NodeProps = {
+  node: FileTree;
+  parentPath: string;
+  currentFile: string;
+  setCurrentFile: (path: string) => void;
+  FileTreeNodeWrapper?: FileTreeNodeWrapper;
+};
 
 const addPath = (parentPath: string, path: string) => {
   if ("" === parentPath) {
@@ -52,16 +64,31 @@ const addPath = (parentPath: string, path: string) => {
   return parentPath + "/" + path;
 };
 
-const TreeNode: FC<NodeProps> = ({ node, parentPath, currentFile, setCurrentFile }) => {
+const TreeNode: FC<NodeProps> = ({ node, parentPath, currentFile, setCurrentFile, FileTreeNodeWrapper }) => {
   const [t] = useTranslation("repos");
 
+  FileTreeNodeWrapper = FileTreeNodeWrapper || (({ children }) => <>{children}</>);
+
+  const label = <div className="ml-1">{node.nodeName}</div>;
+  const icon = <StyledIcon alt={t("diff.showContent")}>folder</StyledIcon>;
   return (
     <li>
       {Object.keys(node.children).length > 0 ? (
         <ul className="py-1 pl-3">
           <li className="is-flex has-text-grey">
-            <StyledIcon alt={t("diff.showContent")}>folder</StyledIcon>
-            <div className="ml-1">{node.nodeName}</div>
+            <FileTreeNodeWrapper
+              path={addPath(parentPath, node.nodeName)}
+              isFile={false}
+              isCurrentFile={false}
+              name={node.nodeName}
+              iconName={"folder"}
+              iconColor={"grey"}
+              originalIcon={icon}
+              originalLabel={label}
+            >
+              {icon}
+              {label}
+            </FileTreeNodeWrapper>
           </li>
           {Object.keys(node.children).map((key) => (
             <TreeNode
@@ -70,23 +97,25 @@ const TreeNode: FC<NodeProps> = ({ node, parentPath, currentFile, setCurrentFile
               parentPath={addPath(parentPath, node.nodeName)}
               currentFile={currentFile}
               setCurrentFile={setCurrentFile}
+              FileTreeNodeWrapper={FileTreeNodeWrapper}
             />
           ))}
         </ul>
       ) : (
         <TreeFile
-          changeType={node.changeType.toLowerCase() as ChangeType}
+          changeType={node.changeType.toLowerCase() as FileChangeType}
           path={node.nodeName}
           parentPath={parentPath}
           currentFile={currentFile}
           setCurrentFile={setCurrentFile}
+          FileTreeNodeWrapper={FileTreeNodeWrapper}
         />
       )}
     </li>
   );
 };
 
-const getColor = (changeType: ChangeType) => {
+const getColor = (changeType: FileChangeType) => {
   switch (changeType) {
     case "add":
       return "success";
@@ -99,7 +128,7 @@ const getColor = (changeType: ChangeType) => {
   }
 };
 
-const getIcon = (changeType: ChangeType) => {
+const getIcon = (changeType: FileChangeType) => {
   switch (changeType) {
     case "add":
     case "copy":
@@ -113,14 +142,22 @@ const getIcon = (changeType: ChangeType) => {
 };
 
 type FileProps = {
-  changeType: ChangeType;
+  changeType: FileChangeType;
   path: string;
   parentPath: string;
   currentFile: string;
   setCurrentFile: (path: string) => void;
+  FileTreeNodeWrapper: FileTreeNodeWrapper;
 };
 
-const TreeFile: FC<FileProps> = ({ changeType, path, parentPath, currentFile, setCurrentFile }) => {
+const TreeFile: FC<FileProps> = ({
+  changeType,
+  path,
+  parentPath,
+  currentFile,
+  setCurrentFile,
+  FileTreeNodeWrapper,
+}) => {
   const [t] = useTranslation("repos");
   const completePath = addPath(parentPath, path);
 
@@ -128,35 +165,50 @@ const TreeFile: FC<FileProps> = ({ changeType, path, parentPath, currentFile, se
     return currentFile === completePath;
   };
 
+  const iconName = getIcon(changeType);
+
+  const icon = (
+    <StackedSpan className="fa-stack">
+      <StyledIcon
+        className={classNames("fa-stack-2x", `has-text-${getColor(changeType)}`)}
+        key={completePath + "file"}
+        type="fas"
+        alt={t("diff.showContent")}
+      >
+        file
+      </StyledIcon>
+      <StyledIcon
+        className={classNames("fa-stack-1x", "is-relative", "has-text-secondary-least")}
+        isSmaller={iconName === "circle"}
+        key={changeType}
+        alt={t(`diff.changes.${changeType}`)}
+      >
+        {iconName}
+      </StyledIcon>
+    </StackedSpan>
+  );
+  const label = <div className={classNames("ml-1", isCurrentFile() ? "has-text-weight-bold" : "")}>{path}</div>;
+
   return (
     <Link
       className="is-flex py-1 pl-3 has-cursor-pointer"
       onClick={() => setCurrentFile(completePath)}
       to={`#diff-${encodeURIComponent(completePath)}`}
     >
-      <StackedSpan className="fa-stack">
-        <StyledIcon
-          className={classNames("fa-stack-2x", `has-text-${getColor(changeType)}`)}
-          key={completePath + "file"}
-          type={isCurrentFile() ? "fas" : "far"}
-          alt={t("diff.showContent")}
-        >
-          file
-        </StyledIcon>
-        <StyledIcon
-          className={classNames(
-            "fa-stack-1x",
-            "is-relative",
-            isCurrentFile() ? "has-text-secondary-least" : `has-text-${getColor(changeType)}`
-          )}
-          isSmaller={getIcon(changeType) === "circle"}
-          key={changeType}
-          alt={t(`diff.changes.${changeType}`)}
-        >
-          {getIcon(changeType)}
-        </StyledIcon>
-      </StackedSpan>
-      <div className="ml-1">{path}</div>
+      <FileTreeNodeWrapper
+        name={path}
+        path={completePath}
+        changeType={changeType}
+        isFile={true}
+        iconName={iconName}
+        iconColor={getColor(changeType)}
+        originalIcon={icon}
+        originalLabel={label}
+        isCurrentFile={isCurrentFile()}
+      >
+        {icon}
+        {label}
+      </FileTreeNodeWrapper>
     </Link>
   );
 };