diff --git a/frontend/src/components/tools/removeBlanks/RemoveBlanksSettings.tsx b/frontend/src/components/tools/removeBlanks/RemoveBlanksSettings.tsx index 0806423c5..198c12674 100644 --- a/frontend/src/components/tools/removeBlanks/RemoveBlanksSettings.tsx +++ b/frontend/src/components/tools/removeBlanks/RemoveBlanksSettings.tsx @@ -19,7 +19,7 @@ const RemoveBlanksSettings = ({ parameters, onParameterChange, disabled = false label={t('removeBlanks.threshold.label', 'Pixel Whiteness Threshold')} value={parameters.threshold} onChange={(v) => onParameterChange('threshold', typeof v === 'string' ? Number(v) : v)} - unit={t('removeBlanks.threshold.unit', '')} + unit='' min={0} max={255} disabled={disabled} @@ -31,7 +31,7 @@ const RemoveBlanksSettings = ({ parameters, onParameterChange, disabled = false label={t('removeBlanks.whitePercent.label', 'White Percent')} value={parameters.whitePercent} onChange={(v) => onParameterChange('whitePercent', typeof v === 'string' ? Number(v) : v)} - unit={t('removeBlanks.whitePercent.unit', '%')} + unit='%' min={0.1} max={100} disabled={disabled} diff --git a/frontend/src/hooks/tools/removeBlanks/useRemoveBlanksOperation.ts b/frontend/src/hooks/tools/removeBlanks/useRemoveBlanksOperation.ts index 4ffc7c90e..479132d6b 100644 --- a/frontend/src/hooks/tools/removeBlanks/useRemoveBlanksOperation.ts +++ b/frontend/src/hooks/tools/removeBlanks/useRemoveBlanksOperation.ts @@ -19,7 +19,6 @@ export const removeBlanksOperationConfig = { buildFormData: buildRemoveBlanksFormData, operationType: 'remove-blanks', endpoint: '/api/v1/misc/remove-blanks', - filePrefix: 'noblank_', defaultParameters, } as const satisfies ToolOperationConfig; @@ -35,7 +34,6 @@ export const useRemoveBlanksOperation = () => { return useToolOperation({ ...removeBlanksOperationConfig, responseHandler, - filePrefix: t('removeBlanks.filenamePrefix', 'noblank') + '_', getErrorMessage: createStandardErrorHandler( t('removeBlanks.error.failed', 'Failed to remove blank pages') ) diff --git a/frontend/src/hooks/tools/removePages/useRemovePagesOperation.ts b/frontend/src/hooks/tools/removePages/useRemovePagesOperation.ts index 118277877..5cf1e6de2 100644 --- a/frontend/src/hooks/tools/removePages/useRemovePagesOperation.ts +++ b/frontend/src/hooks/tools/removePages/useRemovePagesOperation.ts @@ -17,7 +17,6 @@ export const removePagesOperationConfig = { buildFormData: buildRemovePagesFormData, operationType: 'remove-pages', endpoint: '/api/v1/general/remove-pages', - filePrefix: 'removed_pages_', defaultParameters, } as const satisfies ToolOperationConfig; @@ -34,7 +33,6 @@ export const useRemovePagesOperation = () => { return useToolOperation({ ...removePagesOperationConfig, responseHandler, - filePrefix: t('removePages.filenamePrefix', 'removed_pages') + '_', getErrorMessage: createStandardErrorHandler( t('removePages.error.failed', 'Failed to remove pages') ) diff --git a/frontend/src/tools/RemoveBlanks.tsx b/frontend/src/tools/RemoveBlanks.tsx index 89b0a84fb..d7478bdac 100644 --- a/frontend/src/tools/RemoveBlanks.tsx +++ b/frontend/src/tools/RemoveBlanks.tsx @@ -1,4 +1,3 @@ -import { useState, useEffect } from "react"; import { useTranslation } from "react-i18next"; import { createToolFlow } from "../components/tools/shared/createToolFlow"; import { BaseToolProps, ToolComponent } from "../types/tool"; @@ -19,23 +18,6 @@ const RemoveBlanks = (props: BaseToolProps) => { props ); - // Step expansion state management - const [expandedStep, setExpandedStep] = useState<"files" | "settings" | null>("files"); - - // Auto-expand settings when files are selected - useEffect(() => { - if (base.selectedFiles.length > 0 && expandedStep === "files") { - setExpandedStep("settings"); - } - }, [base.selectedFiles.length, expandedStep]); - - // Collapse all steps when results appear - useEffect(() => { - if (base.hasResults) { - setExpandedStep(null); - } - }, [base.hasResults]); - const settingsContent = ( { const handleSettingsClick = () => { if (base.hasResults) { base.handleSettingsReset(); - } else { - if (!base.hasFiles) return; - setExpandedStep(expandedStep === "settings" ? null : "settings"); } }; @@ -61,7 +40,7 @@ const RemoveBlanks = (props: BaseToolProps) => { steps: [ { title: t("removeBlanks.settings.title", "Settings"), - isCollapsed: expandedStep !== "settings", + isCollapsed: base.settingsCollapsed, onCollapsedClick: handleSettingsClick, content: settingsContent, tooltip: tooltipContent,