refactor(pres_mgmt): menu consistency — modal pattern, prop cleanup, styling
All 6 *_page_menu.svelte files now consistently use:
- Flowbite Modal (not expandable panels) with matching header/close pattern
- interface Props (no export keyword) with only props actually used
- flex flex-row flex-wrap gap-0.5 on ae_menu__object_options span
Removed unused props and their call-site bindings:
- location_page_menu: removed data prop; +page.svelte drops data={data}
- locations_page_menu: removed unused log_lvl prop
- presenter_page_menu: removed data prop; +page.svelte drops {data}
- event_reports_page_menu: removed data prop; +page.svelte drops {lq__event_obj} data pass
event_page_menu: fixed interface export keyword (was export interface)
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -70,7 +70,7 @@
|
||||
|
||||
<span class="ae_menu__object_options flex flex-row flex-wrap gap-0.5 items-center justify-around">
|
||||
<!-- Event Files toggle — the one real working button from the old menu, kept in place -->
|
||||
<button
|
||||
<!-- <button
|
||||
type="button"
|
||||
onclick={() => {
|
||||
$events_loc.pres_mgmt.show_content__event_view =
|
||||
@@ -98,7 +98,7 @@
|
||||
{$lq__event_obj?.file_count}×
|
||||
</span>
|
||||
{/if}
|
||||
</button>
|
||||
</button> -->
|
||||
|
||||
<!-- Options modal trigger -->
|
||||
{#if $ae_loc.trusted_access}
|
||||
|
||||
@@ -105,7 +105,6 @@
|
||||
|
||||
<section class="ae_events_pres_mgmt_event_location container mx-auto py-1 px-2 pb-16 space-y-6">
|
||||
<Location_page_menu
|
||||
data={data}
|
||||
lq__event_location_obj={lq__event_location_obj}
|
||||
lq__event_obj={lq__event_obj}
|
||||
/>
|
||||
|
||||
@@ -1,13 +1,11 @@
|
||||
<script lang="ts">
|
||||
export interface Props {
|
||||
data?: any;
|
||||
interface Props {
|
||||
log_lvl?: number;
|
||||
lq__event_obj: any;
|
||||
lq__event_location_obj: any;
|
||||
}
|
||||
|
||||
let {
|
||||
data = null,
|
||||
log_lvl = 0,
|
||||
lq__event_obj,
|
||||
lq__event_location_obj
|
||||
@@ -71,7 +69,7 @@
|
||||
events__session_search={$events_slct.event_id}
|
||||
/>
|
||||
|
||||
<span class="ae_menu__object_options flex flex-row flex-wrap items-center justify-around">
|
||||
<span class="ae_menu__object_options flex flex-row flex-wrap gap-0.5 items-center justify-around">
|
||||
<!-- Options modal trigger -->
|
||||
{#if $ae_loc.trusted_access}
|
||||
<button
|
||||
|
||||
@@ -1,10 +1,9 @@
|
||||
<script lang="ts">
|
||||
interface Props {
|
||||
log_lvl?: number;
|
||||
lq__event_obj: any;
|
||||
}
|
||||
|
||||
let { log_lvl = 0, lq__event_obj }: Props = $props();
|
||||
let { lq__event_obj }: Props = $props();
|
||||
|
||||
import { Modal } from 'flowbite-svelte';
|
||||
import { Settings, X, Info } from '@lucide/svelte';
|
||||
|
||||
@@ -138,7 +138,7 @@
|
||||
max-w-max
|
||||
"
|
||||
>
|
||||
<Presenter_page_menu {data} {lq__event_obj} {lq__event_presenter_obj} />
|
||||
<Presenter_page_menu {lq__event_obj} {lq__event_presenter_obj} />
|
||||
|
||||
{#if !$lq__event_presenter_obj}
|
||||
<div>
|
||||
|
||||
@@ -1,13 +1,11 @@
|
||||
<script lang="ts">
|
||||
interface Props {
|
||||
data?: any;
|
||||
log_lvl?: number;
|
||||
lq__event_obj: any;
|
||||
lq__event_presenter_obj: any;
|
||||
}
|
||||
|
||||
let {
|
||||
data = null,
|
||||
log_lvl = 0,
|
||||
lq__event_obj,
|
||||
lq__event_presenter_obj
|
||||
|
||||
@@ -105,7 +105,7 @@
|
||||
</title>
|
||||
</svelte:head>
|
||||
|
||||
<Event_reports_page_menu {data} {lq__event_obj} />
|
||||
<Event_reports_page_menu {lq__event_obj} />
|
||||
|
||||
{#if !$lq__event_obj}
|
||||
<div>
|
||||
|
||||
@@ -1,10 +1,9 @@
|
||||
<script lang="ts">
|
||||
interface Props {
|
||||
data: any;
|
||||
lq__event_obj: any;
|
||||
}
|
||||
|
||||
let { data, lq__event_obj }: Props = $props();
|
||||
let { lq__event_obj }: Props = $props();
|
||||
|
||||
import { Modal } from 'flowbite-svelte';
|
||||
import { Settings, X, Info } from '@lucide/svelte';
|
||||
@@ -33,7 +32,7 @@
|
||||
events__session_search={$events_slct.event_id}
|
||||
/>
|
||||
|
||||
<span class="ae_menu__object_options flex flex-row items-center justify-around">
|
||||
<span class="ae_menu__object_options flex flex-row flex-wrap gap-0.5 items-center justify-around">
|
||||
<!-- Options modal trigger -->
|
||||
{#if $ae_loc.trusted_access}
|
||||
<button
|
||||
|
||||
Reference in New Issue
Block a user