mirror of
https://github.com/suitenumerique/drive.git
synced 2026-04-25 17:15:19 +02:00
♻️(frontend) remove unused props and CSS in preview viewers
Remove the unused isSidebarOpen prop from PdfControls and strip unnecessary position/min-height rules from the video player that were left over from earlier layout iterations.
This commit is contained in:
@@ -5,7 +5,6 @@ import { ZoomControls } from "../../components/ZoomControls/ZoomControls";
|
||||
interface PdfControlsProps {
|
||||
numPages: number;
|
||||
pageInputValue: string;
|
||||
isSidebarOpen: boolean;
|
||||
onToggleSidebar: () => void;
|
||||
onPageInputChange: (e: React.ChangeEvent<HTMLInputElement>) => void;
|
||||
onPageInputSubmit: () => void;
|
||||
|
||||
@@ -171,7 +171,6 @@ export function PdfPreview({
|
||||
<PdfControls
|
||||
numPages={numPages}
|
||||
pageInputValue={pageInputValue}
|
||||
isSidebarOpen={isSidebarOpen}
|
||||
onToggleSidebar={toggleSidebar}
|
||||
onPageInputChange={handlePageInputChange}
|
||||
onPageInputSubmit={handlePageInputSubmit}
|
||||
|
||||
@@ -1,5 +1,4 @@
|
||||
.video-player {
|
||||
position: relative;
|
||||
overflow: hidden;
|
||||
width: 100%;
|
||||
height: 100vh; // Use full available height
|
||||
@@ -69,7 +68,6 @@
|
||||
}
|
||||
|
||||
&__controls {
|
||||
position: relative;
|
||||
border-top: none;
|
||||
padding: 16px 0;
|
||||
transition: opacity 0.3s ease;
|
||||
@@ -78,7 +76,6 @@
|
||||
flex-direction: column;
|
||||
gap: var(--c--globals--spacings--md);
|
||||
flex-shrink: 0; // Prevent controls from shrinking
|
||||
min-height: 120px; // Minimum height for controls
|
||||
}
|
||||
}
|
||||
|
||||
@@ -97,7 +94,6 @@
|
||||
}
|
||||
|
||||
&__controls {
|
||||
min-height: 100px;
|
||||
padding: 12px;
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user