style(role-parity): align runtime role list actions with next surface
This commit is contained in:
parent
78d03e67fb
commit
cd780906b1
1 changed files with 2 additions and 30 deletions
|
|
@ -1,5 +1,5 @@
|
||||||
import { A, useSearchParams } from '@solidjs/router';
|
import { A, useSearchParams } from '@solidjs/router';
|
||||||
import { createMemo, createResource, createSignal, Show } from 'solid-js';
|
import { createMemo, createResource, Show } from 'solid-js';
|
||||||
import AdminShell from '~/components/AdminShell';
|
import AdminShell from '~/components/AdminShell';
|
||||||
import ExternalRoleTabs from '~/components/admin/ExternalRoleTabs';
|
import ExternalRoleTabs from '~/components/admin/ExternalRoleTabs';
|
||||||
|
|
||||||
|
|
@ -37,25 +37,9 @@ async function loadExternalRoles(): Promise<ExternalRole[]> {
|
||||||
|
|
||||||
export default function RuntimeRolesPage() {
|
export default function RuntimeRolesPage() {
|
||||||
const [searchParams] = useSearchParams();
|
const [searchParams] = useSearchParams();
|
||||||
const [roles, { refetch }] = createResource(loadExternalRoles);
|
const [roles] = createResource(loadExternalRoles);
|
||||||
const [deleting, setDeleting] = createSignal('');
|
|
||||||
const [deleteError, setDeleteError] = createSignal('');
|
|
||||||
const selectedRoleKey = createMemo(() => (searchParams.roleKey || '').toLowerCase());
|
const selectedRoleKey = createMemo(() => (searchParams.roleKey || '').toLowerCase());
|
||||||
|
|
||||||
const handleDelete = async (id: string, name: string) => {
|
|
||||||
if (!confirm(`Delete external role "${name}"?`)) return;
|
|
||||||
try {
|
|
||||||
setDeleting(id);
|
|
||||||
const res = await fetch(`${API}/api/admin/roles/${id}`, { method: 'DELETE' });
|
|
||||||
if (!res.ok) throw new Error('Failed to delete');
|
|
||||||
refetch();
|
|
||||||
} catch (err: any) {
|
|
||||||
setDeleteError(err.message || 'Failed to delete');
|
|
||||||
} finally {
|
|
||||||
setDeleting('');
|
|
||||||
}
|
|
||||||
};
|
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<AdminShell>
|
<AdminShell>
|
||||||
<div class="page-hero-card page-actions">
|
<div class="page-hero-card page-actions">
|
||||||
|
|
@ -71,10 +55,6 @@ export default function RuntimeRolesPage() {
|
||||||
|
|
||||||
<ExternalRoleTabs />
|
<ExternalRoleTabs />
|
||||||
|
|
||||||
<Show when={deleteError()}>
|
|
||||||
<div class="error-box">{deleteError()}</div>
|
|
||||||
</Show>
|
|
||||||
|
|
||||||
<section class="card" style="padding: 0; overflow: hidden;">
|
<section class="card" style="padding: 0; overflow: hidden;">
|
||||||
<div style="display:flex;align-items:center;justify-content:space-between;padding:16px 20px;border-bottom:1px solid #e2e8f0">
|
<div style="display:flex;align-items:center;justify-content:space-between;padding:16px 20px;border-bottom:1px solid #e2e8f0">
|
||||||
<div>
|
<div>
|
||||||
|
|
@ -128,14 +108,6 @@ export default function RuntimeRolesPage() {
|
||||||
<div class="table-actions">
|
<div class="table-actions">
|
||||||
<A class="action-icon-btn" href={`/admin/runtime-roles/${encodeURIComponent(role.roleKey)}`} target="_blank" rel="noreferrer" title="View External Role">👁</A>
|
<A class="action-icon-btn" href={`/admin/runtime-roles/${encodeURIComponent(role.roleKey)}`} target="_blank" rel="noreferrer" title="View External Role">👁</A>
|
||||||
<A class="action-icon-btn" href={`/admin/runtime-roles/${encodeURIComponent(role.roleKey)}`} title="Edit External Role">✎</A>
|
<A class="action-icon-btn" href={`/admin/runtime-roles/${encodeURIComponent(role.roleKey)}`} title="Edit External Role">✎</A>
|
||||||
<button
|
|
||||||
class="action-icon-btn danger"
|
|
||||||
disabled={deleting() === role.id}
|
|
||||||
onClick={() => handleDelete(role.id, role.displayName)}
|
|
||||||
title="Delete External Role"
|
|
||||||
>
|
|
||||||
{deleting() === role.id ? '...' : '🗑'}
|
|
||||||
</button>
|
|
||||||
</div>
|
</div>
|
||||||
</td>
|
</td>
|
||||||
</tr>
|
</tr>
|
||||||
|
|
|
||||||
Loading…
Add table
Reference in a new issue