Skip to content

fix(viz): hide controls in static docking boxes (DDOS-7327) - #609

Merged
sg-s merged 2 commits into
mainfrom
fix/DDOS-7327-hide-static-docking-controls
Aug 20, 2026
Merged

fix(viz): hide controls in static docking boxes (DDOS-7327)#609
sg-s merged 2 commits into
mainfrom
fix/DDOS-7327-hide-static-docking-controls

Conversation

@sg-s

@sg-s sg-s commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Request a read-only Mol* docking box viewer for static show_box() renders and pose overlays.
  • Preserve editing and Python synchronization for show_box(interactive=True).
  • Prevent visual-only edits that look committed but cannot reach Python.

Depends on platform-ui#1490 and its hosted Mol* bundle deployment.

Jira: DDOS-7327

Test plan

  • uv run --extra lint ruff format src/viz/molstar_html.py tests/test_molstar_html.py tests/test_docking_interactive_box.py
  • uv run --extra lint ruff check --select I src/viz/molstar_html.py tests/test_molstar_html.py tests/test_docking_interactive_box.py --fix
  • uv run --extra test --extra core --extra tools --extra plots pytest --env local -x tests/test_molstar_html.py tests/test_docking_interactive_box.py -q (43 passed)

Made with Cursor

sg-s and others added 2 commits August 20, 2026 12:47
Prevent static previews from presenting edits that cannot synchronize back to Python.

Co-authored-by: Cursor <cursoragent@cursor.com>
Undo local notebook promotion performed by the commit hook so this PR stays focused on static viewer controls.

Co-authored-by: Cursor <cursoragent@cursor.com>
@sg-s
sg-s requested a review from a team as a code owner August 20, 2026 16:49
@sg-s
sg-s deployed to staging August 20, 2026 16:49 — with GitHub Actions Active
@sg-s
sg-s merged commit 94d799a into main Aug 20, 2026
6 of 11 checks passed
@sg-s
sg-s deleted the fix/DDOS-7327-hide-static-docking-controls branch August 20, 2026 20:02
@sg-s sg-s self-assigned this Aug 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant