fix: improve App Config slide-in panel layout
- Narrowed panel from 640px to 520px so main table columns stay visible - Settings grid uses CSS grid (3 columns) for proper wrapping - Removed unused PanelActions component that caused white footer bar Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -1,5 +1,5 @@
|
|||||||
.widePanel {
|
.widePanel {
|
||||||
width: 640px !important;
|
width: 520px !important;
|
||||||
}
|
}
|
||||||
|
|
||||||
.panelSection {
|
.panelSection {
|
||||||
@@ -16,9 +16,9 @@
|
|||||||
}
|
}
|
||||||
|
|
||||||
.settingsGrid {
|
.settingsGrid {
|
||||||
display: flex;
|
display: grid;
|
||||||
flex-wrap: wrap;
|
grid-template-columns: 1fr 1fr 1fr;
|
||||||
gap: 16px;
|
gap: 12px;
|
||||||
}
|
}
|
||||||
|
|
||||||
.field {
|
.field {
|
||||||
|
|||||||
@@ -284,22 +284,9 @@ export default function AppConfigPage() {
|
|||||||
onClose={() => setSelectedApp(null)}
|
onClose={() => setSelectedApp(null)}
|
||||||
title={selectedApp ?? ''}
|
title={selectedApp ?? ''}
|
||||||
className={styles.widePanel}
|
className={styles.widePanel}
|
||||||
actions={selectedApp ? <PanelActions appId={selectedApp} onClose={() => setSelectedApp(null)} /> : undefined}
|
|
||||||
>
|
>
|
||||||
{selectedApp && <AppConfigDetail appId={selectedApp} onClose={() => setSelectedApp(null)} />}
|
{selectedApp && <AppConfigDetail appId={selectedApp} onClose={() => setSelectedApp(null)} />}
|
||||||
</DetailPanel>
|
</DetailPanel>
|
||||||
</div>
|
</div>
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
function PanelActions({ appId, onClose }: { appId: string; onClose: () => void }) {
|
|
||||||
// Edit/Save/Cancel actions rendered in the panel footer
|
|
||||||
const { data: config } = useApplicationConfig(appId);
|
|
||||||
const updateConfig = useUpdateApplicationConfig();
|
|
||||||
const { toast } = useToast();
|
|
||||||
const [editing, setEditing] = useState(false);
|
|
||||||
|
|
||||||
// This is a simplified stub — the actual editing state is managed inside AppConfigDetail.
|
|
||||||
// For now, the actions slot is reserved for future use.
|
|
||||||
return null;
|
|
||||||
}
|
|
||||||
|
|||||||
Reference in New Issue
Block a user