refactor(hub): simplify iframe address bar options

Author: dvcolomban

docs/content/8.references/6.hub-api.md

@@ -137,7 +137,7 @@ The built-in variants of the open dock union (`DevframeDockEntryRegistry`, `@dev
 
 | Type | The hub UI provider renders |
 |---|---|
-| `iframe` | the entry's `url` in a kept-alive iframe (per `frameId` when shared); honor `subTabs` soft nav; force the existing address bar with `showAddressBar`; override Back, Reload, and Open externally visibility with `controls.back`, `controls.reload`, and `controls.openExternal` |
+| `iframe` | the entry's `url` in a kept-alive iframe (per `frameId` when shared); honor `subTabs` soft nav; show the existing address bar with `addressBar: true`, or configure Back, Reload, and Open externally by passing an `addressBar` object |
 | `action` | a dock-rail button; activating runs its client script |
 | `custom-render` | a container its client script mounts into |
 | `launcher` | a launch call-to-action reflecting `launcher.status` |

packages/hub-ui/src/client/components/views/ViewIframe.vue

@@ -20,7 +20,8 @@ const props = defineProps<{
 
 const settings = useSettings(props.context)
 const isEdgeMode = computed(() => props.context.panel.store.mode === 'edge')
-const showAddressBar = computed(() => settings.value.showIframeAddressBar || props.entry.showAddressBar === true)
+const addressBarControls = computed(() => typeof props.entry.addressBar === 'object' ? props.entry.addressBar : undefined)
+const showAddressBar = computed(() => settings.value.showIframeAddressBar || Boolean(props.entry.addressBar))
 
 const ADDRESS_BAR_HEIGHT = 40
 
@@ -83,9 +84,9 @@ const isCrossOrigin = computed(() => {
     return true // Assume cross-origin if URL parsing fails
   }
 })
-const showBack = computed(() => props.entry.controls?.back ?? !isCrossOrigin.value)
-const showReload = computed(() => props.entry.controls?.reload ?? !isCrossOrigin.value)
-const showOpenExternal = computed(() => props.entry.controls?.openExternal ?? false)
+const showBack = computed(() => addressBarControls.value?.back ?? !isCrossOrigin.value)
+const showReload = computed(() => addressBarControls.value?.reload ?? !isCrossOrigin.value)
+const showOpenExternal = computed(() => addressBarControls.value?.openExternal ?? false)
 
 // Display URL - hides host if same as current page. The remote connection
 // descriptor is stripped so its auth token can't be read (or copied) out of the

packages/hub/src/types/docks.ts

@@ -203,15 +203,13 @@ declare module 'devframe/types' {
 export interface DevframeViewIframe extends DevframeDockEntryBase {
   type: 'iframe'
   url: string
-  /** Request the address bar for this iframe dock. The user's global always-show setting forces it on for every iframe. */
-  showAddressBar?: boolean
-  /** Optional address-bar controls. Omitted values retain the existing origin-based behavior. */
-  controls?: {
+  /** Request the address bar for this iframe dock. Pass an object to configure its controls. The user's global always-show setting forces it on for every iframe. */
+  addressBar?: boolean | {
     /** Override Back visibility. Cross-origin history access may still be blocked by the browser. */
     back?: boolean
     /** Override Reload visibility. */
     reload?: boolean
-    /** Override the built-in action that opens the displayed HTTP(S) URL in a new tab. */
+    /** Override Open externally visibility. */
     openExternal?: boolean
   }
   /**

tests/__snapshots__/tsnapi/@devframes/hub/index.snapshot.d.ts

@@ -293,8 +293,7 @@ export interface DevframeViewGroup extends DevframeDockEntryBase {
 export interface DevframeViewIframe extends DevframeDockEntryBase {
   type: 'iframe';
   url: string;
-  showAddressBar?: boolean;
-  controls?: {
+  addressBar?: boolean | {
     back?: boolean;
     reload?: boolean;
     openExternal?: boolean;