fix(trackerobjects): de-squeeze library row when binding button is added

Two cumulative changes to keep the four-button row readable:

- Skip the assign-tracker button entirely for SpawnMode.Scene and
  SpawnMethod.Embedded rows. A disabled fourth button just pushed the
  Select/Teleport/Remove labels off the right of those rows, and those
  instances can't host a tracker binding anyway (scene spawn isn't
  user-owned; embedded has no pickup/rigid surface).
- Shorten the labels from "Assign Tracker" / "Unbind Tracker" to
  "Assign" / "Unbind". Matches the brevity of Select/Remove and frees
  horizontal space on rows that DO get the button. The prop-row
  context already makes "what's being assigned" obvious.

REQUIREMENTS.md and the integration README updated to match.

Author: towneh

Basis/Packages/com.basis.framework/BasisUI/Localization/Languages/en.json

@@ -344,8 +344,8 @@
     { "key": "library.share", "value": "Share" },
     { "key": "library.teleportTo", "value": "Teleport To" },
     { "key": "library.remove", "value": "Remove" },
-    { "key": "library.assignTracker", "value": "Assign Tracker" },
-    { "key": "library.unbindTracker", "value": "Unbind Tracker" },
+    { "key": "library.assignTracker", "value": "Assign" },
+    { "key": "library.unbindTracker", "value": "Unbind" },
     { "key": "library.trackerPicker.title", "value": "Choose Tracker" },
     { "key": "library.trackerPicker.empty", "value": "No trackers are currently connected." },
     { "key": "library.trackerPicker.confirm", "value": "Bind" },

Basis/Packages/com.basis.integration.trackerobjects/README.md

@@ -1,6 +1,6 @@
 # Basis Tracker Objects Integration
 
-Bridges [`com.basis.trackerobjects`](../com.basis.trackerobjects/REQUIREMENTS.md) into the Basis library menu. When a prop has been instantiated and shows up in the library's instantiated-items tab, this package adds an **Assign Tracker** button to the row. Clicking it opens a tracker picker; confirming a tracker binds the prop's GameObject to that tracker via `BasisTrackerObjectManager`. Clicking **Unbind Tracker** removes the binding.
+Bridges [`com.basis.trackerobjects`](../com.basis.trackerobjects/REQUIREMENTS.md) into the Basis library menu. When a prop has been instantiated and shows up in the library's instantiated-items tab, this package adds an **Assign** button to the row. Clicking it opens a tracker picker; confirming a tracker binds the prop's GameObject to that tracker via `BasisTrackerObjectManager`. Clicking **Unbind** removes the binding.
 
 ## Why a separate package
 
@@ -10,13 +10,13 @@ Bridges [`com.basis.trackerobjects`](../com.basis.trackerobjects/REQUIREMENTS.md
 
 - A `[RuntimeInitializeOnLoadMethod(SubsystemRegistration)]` subscriber on `LibraryProvider.OnInstanceRowCreated`. For every instantiated-object row that `LibraryProvider` builds, this appends a `StandardButton` between the existing Select and Teleport buttons.
 - The button is non-interactable for `SpawnMode.Scene` and `SpawnMethod.Embedded` items — same rule the Select button applies.
-- Clicking **Assign Tracker** opens a `DialogBox<BasisInput>` modal listing the currently-connected trackers eligible for prop binding. Confirming a row calls `BasisTrackerObjectManager.TryCreateBinding` with the spawn instance's `LoadedNetID` and `GameObject` transform. The picker excludes:
+- Clicking **Assign** opens a `DialogBox<BasisInput>` modal listing the currently-connected trackers eligible for prop binding. Confirming a row calls `BasisTrackerObjectManager.TryCreateBinding` with the spawn instance's `LoadedNetID` and `GameObject` transform. The picker excludes:
   - `BasisVirtualMidpointInput` instances (the virtual half of an active pair).
   - Trackers with `BasisInput.IsLinked == true` (one half of an active pair).
   - Trackers whose `UniqueDeviceIdentifier` has a `BasisTrackerRoleOverride.TryGetOverride` hit.
   - Devices the input matcher has pinned to a fixed role (HMD, named controllers, etc.).
   - Trackers currently driving an avatar bone via calibration. Decalibrate first if you want to reuse a calibrated tracker for a prop.
-- Clicking **Unbind Tracker** calls `BasisTrackerObjectManager.TryRemoveBinding` directly — no confirmation dialog. Unbind isn't destructive; the binding just lifts.
+- Clicking **Unbind** calls `BasisTrackerObjectManager.TryRemoveBinding` directly — no confirmation dialog. Unbind isn't destructive; the binding just lifts.
 
 ## Compile guards
 

Basis/Packages/com.basis.integration.trackerobjects/Runtime/BasisTrackerObjectsLibraryHook.cs

@@ -8,7 +8,6 @@
 using Basis.Scripts.TransformBinders.BoneControl;
 using Basis.TrackerObjects;
 using UnityEngine;
-using UnityEngine.UI;
 
 namespace Basis.Integration.TrackerObjects
 {
@@ -31,20 +30,18 @@ private static void OnRowCreated(RectTransform parent, BasisRuntimeSpawnRegistry
             string netID = instance.LoadedNetID;
             if (string.IsNullOrEmpty(netID)) return;
 
+            // Scene-mode and embedded instances can't host a tracker binding (no
+            // pickup/rigid surface to drive, and they're not user-owned spawns), so
+            // skip adding the button at all — a disabled fourth button just pushes
+            // the Select/Teleport/Remove row over.
+            if (instance.SpawnMode == BasisRuntimeSpawnRegistry.SpawnMode.Scene) return;
+            if (instance.SpawnMethod == BasisRuntimeSpawnRegistry.SpawnMethod.Embedded) return;
+
             bool hasBinding = BasisTrackerObjectManager.TryGetBindingByLoadedNetID(netID, out _);
             PanelButton button = PanelButton.CreateNew(PanelButton.ButtonStyles.StandardButton, parent);
             button.Descriptor.SetTitle(BasisLocalization.Get(hasBinding ? "library.unbindTracker" : "library.assignTracker"));
             button.SetSize(RowSize);
 
-            bool interactable =
-                instance.SpawnMode != BasisRuntimeSpawnRegistry.SpawnMode.Scene
-                && instance.SpawnMethod != BasisRuntimeSpawnRegistry.SpawnMethod.Embedded;
-            if (button.Descriptor.gameObject.TryGetComponent(out Button buttonComponent))
-            {
-                buttonComponent.interactable = interactable;
-            }
-            if (!interactable) return;
-
             button.OnClicked += async () =>
             {
                 if (BasisTrackerObjectManager.TryGetBindingByLoadedNetID(netID, out BasisTrackerBinding existing))

Basis/Packages/com.basis.trackerobjects/REQUIREMENTS.md

@@ -77,12 +77,14 @@ A new `PanelButton` is added to each instantiated-object row in `LibraryProvider
 
 | Binding state for this `InstanceId` | Button style       | Button label (localization key)        |
 | ----------------------------------- | ------------------ | -------------------------------------- |
-| No binding                          | `StandardButton`   | `library.assignTracker` ("Assign Tracker") |
-| Binding exists                      | `StandardButton`   | `library.unbindTracker` ("Unbind Tracker") |
+| No binding                          | `StandardButton`   | `library.assignTracker` ("Assign")     |
+| Binding exists                      | `StandardButton`   | `library.unbindTracker` ("Unbind")     |
+
+Labels are deliberately one word — the prop-row context already makes "what's being assigned" obvious, and the row's equal-share horizontal layout means a longer label would squeeze Select/Teleport/Remove visibly.
 
 The style stays fixed; only the label flips. This matches the existing Select/Deselect pattern on the same row and avoids signalling unbind as destructive (it isn't — no data loss, the binding just lifts).
 
-The button is non-interactable when the instance's `SpawnMode == Scene` or `SpawnMethod == Embedded`, mirroring the existing Select button's interactability rules.
+The button is skipped entirely (not added to the row) when the instance's `SpawnMode == Scene` or `SpawnMethod == Embedded`. A disabled fourth button would push Select/Teleport/Remove off the right of the row, and those instances can't host a tracker binding regardless — a scene spawn isn't user-owned and an embedded one has no pickup/rigid surface to drive.
 
 ### Tracker picker dialog
 
@@ -100,8 +102,8 @@ The picker is user-initiated, so the host must always show it — if the dialogu
 
 Localization strings land in every `Localization/Languages/*.json` file. English values:
 
-- `library.assignTracker` — "Assign Tracker"
-- `library.unbindTracker` — "Unbind Tracker"
+- `library.assignTracker` — "Assign"
+- `library.unbindTracker` — "Unbind"
 - `library.trackerPicker.title` — "Choose Tracker"
 - `library.trackerPicker.empty` — "No trackers are currently connected."
 - `library.trackerPicker.confirm` — "Bind"