Scott.Idem/OSIT-AE-App-Svelte
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor: rename ae_hide_menu URL param to hide_menu

Browse Source 
ae_ prefix belongs on Svelte component/variable names, not URL params.
Updated both the consumer (+layout.svelte) and the builder (jitsi_url_builder).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Scott Idem
2026-03-17 20:03:28 -04:00
parent 5978c5e341
commit 931df5581f
2 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/routes/+layout.svelte
View File

@@ -254,10 +254,10 @@
if (iframe === 'true') $ae_loc.iframe = true;
else if (iframe === 'false') $ae_loc.iframe = false;
// ae_hide_menu=true — suppress the AE system bar when embedding in Novi or
// hide_menu=true — suppress the AE system bar when embedding in Novi or
// other host pages with their own navigation. Applies even for trusted_access
// users who would otherwise always see the menu in iframe mode.
if (data.url.searchParams.get('ae_hide_menu') === 'true') {
if (data.url.searchParams.get('hide_menu') === 'true') {
$ae_loc.sys_menu.hide = true;
}

4
src/routes/idaa/(idaa)/jitsi_reports/ae_idaa_comp__jitsi_url_builder.svelte
View File

@@ -68,7 +68,7 @@
if (disable_reaction) p.set('reaction_sound', 'true');
if (disable_raise_hand) p.set('raise_hand_sound', 'true');
// AE embed option: hide AE system menu when embedding in other pages
if (hide_ae_menu) p.set('ae_hide_menu', 'true');
if (hide_ae_menu) p.set('hide_menu', 'true');
return `${effective_base}?${p.toString()}`;
});
@@ -231,7 +231,7 @@
/>
<span class="text-xs"
>Hide AE system menu <span class="opacity-40"
>(ae_hide_menu=true)</span
>(hide_menu=true)</span
></span
>
</label>