Fixes ui bug and remove redundant definition

grafana · Jan 23, 2025 · fd1abad · fd1abad
1 parent bb06a25
commit fd1abad
Show file tree

Hide file tree

Showing 6 changed files with 10 additions and 44 deletions.
diff --git a/.gitignore b/.gitignore
@@ -38,6 +38,7 @@ pkg/loki/wal
 tools/lambda-promtail/main
 tools/dev/kafka/data/
 pkg/dataobj/explorer/ui/node_modules/*
+pkg/dataobj/explorer/ui/.vite/*
 
 # Submodule added by `act` CLI
 _shared-workflows-dockerhub-login

diff --git a/...bj/explorer/dist/assets/index-Cw4VlniF.js → ...bj/explorer/dist/assets/index-DFUF28zy.js b/...bj/explorer/dist/assets/index-Cw4VlniF.js → ...bj/explorer/dist/assets/index-DFUF28zy.js
diff --git a/pkg/dataobj/explorer/dist/assets/style-DGrNsbSR.css b/pkg/dataobj/explorer/dist/assets/style-DGrNsbSR.css
diff --git a/pkg/dataobj/explorer/dist/assets/style-Dz5w-Rts.css b/pkg/dataobj/explorer/dist/assets/style-Dz5w-Rts.css
diff --git a/pkg/dataobj/explorer/dist/index.html b/pkg/dataobj/explorer/dist/index.html
@@ -5,8 +5,8 @@
     <meta charset="UTF-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
     <title>DataObj Explorer</title>
-  <script type="module" crossorigin src="/dataobj/explorer/assets/index-Cw4VlniF.js"></script>
-  <link rel="stylesheet" crossorigin href="/dataobj/explorer/assets/style-DGrNsbSR.css">
+  <script type="module" crossorigin src="/dataobj/explorer/assets/index-DFUF28zy.js"></script>
+  <link rel="stylesheet" crossorigin href="/dataobj/explorer/assets/style-Dz5w-Rts.css">
 </head>
 
 <body>

diff --git a/pkg/dataobj/explorer/ui/src/components/file-metadata/FileMetadata.tsx b/pkg/dataobj/explorer/ui/src/components/file-metadata/FileMetadata.tsx
@@ -4,45 +4,10 @@ import { DateWithHover } from "../common/DateWithHover";
 import { Link } from "react-router-dom";
 import { useBasename } from "../../contexts/BasenameContext";
 import { CompressionRatio } from "./CompressionRatio";
-
-interface PageInfo {
-  compressed_size: number;
-  uncompressed_size: number;
-  rows_count: number;
-  values_count: number;
-  encoding: string;
-  data_offset: number;
-  data_size: number;
-}
-
-interface ColumnInfo {
-  name?: string;
-  type: string;
-  value_type: string;
-  rows_count: number;
-  compression: string;
-  uncompressed_size: number;
-  compressed_size: number;
-  metadata_offset: number;
-  metadata_size: number;
-  values_count: number;
-  pages: PageInfo[];
-}
-
-interface SectionMetadata {
-  type: string;
-  totalCompressedSize: number;
-  totalUncompressedSize: number;
-  columnCount: number;
-  columns: ColumnInfo[];
-}
+import { FileMetadataResponse } from "../../types/metadata";
 
 interface FileMetadataProps {
-  metadata: {
-    sections: SectionMetadata[];
-    error?: string;
-    lastModified: string;
-  };
+  metadata: FileMetadataResponse;
   filename: string;
   className?: string;
 }
@@ -218,10 +183,10 @@ export const FileMetadata: React.FC<FileMetadataProps> = ({
 
               {/* Section Content */}
               <div
-                className={`transition-all duration-700 ease-in-out overflow-hidden ${
+                className={`transition-all duration-700 ease-in-out ${
                   expandedSectionIndex === sectionIndex
-                    ? "max-h-[5000px] opacity-100"
-                    : "max-h-0 opacity-0"
+                    ? "opacity-100"
+                    : "opacity-0 hidden"
                 }`}
               >
                 <div className="p-4 bg-gray-50 dark:bg-gray-800">