calesthio/OptionsCanvas

Fixes a regression where the SL/TP price lines on the chart for open positions were visually present but read-only — you couldn't trail your stops without closing and re-entering.

What's fixed

DragHandles (the chart layer that listens for clicks on horizontal price lines and drives the trailing-stop drag UX) was instantiated in main.js but never had .enable() called on it. A comment in the code explained the intent: the chart's order-panel pills had taken over SL/TP setup for new orders, so DragHandles was meant to be "kept available for existing-position management but disabled by default." The disabled-by-default part shipped; the re-enable for open positions did not.

Now DragHandles.enable() is called immediately after construction. The two systems coexist cleanly because DragHandles already scopes its pointerdown handler to ignore clicks inside .order-panel-on-chart (the pill container).

How trailing works now

1. Open a position via the pill or hotkey
2. Solid red SL line + solid green TP line appear on the chart at your levels
3. Grab the solid line itself (not the pill) and drag to a new price
4. A "Set Stop Loss: $X?" popup appears with Confirm / Cancel buttons
5. Click Confirm within 10 seconds → backend updates, positions.stop_loss_price (or take_profit_price) in SQLite reflects the new level
6. The check_stop_loss() / check_take_profit() loops use the new level on the very next poll

Server-side stop guarantee still holds — the trailed level lives in our local DB, the broker order book never sees it until breach.

How to upgrade

git pull
# refresh the browser tab — pure frontend change, no platform restart needed

Verified

• window.DragHandles.enabled flips to true after refresh
• Dragging the SL line on an open TSLA PUT shows the confirm popup → click Confirm → DB stop_loss_price reflects the new value
• TP works identically (drag the green line)
• New-order pill drag for SL/TP setup unchanged