Skip to content

feat(mcq): enlarge and maximize image previews in Gtk dialogs - #32

Merged
ajlennon merged 1 commit into
mainfrom
feat/mcq-image-maximize
Aug 3, 2026
Merged

feat(mcq): enlarge and maximize image previews in Gtk dialogs#32
ajlennon merged 1 commit into
mainfrom
feat/mcq-image-maximize

Conversation

@ajlennon

@ajlennon ajlennon commented Aug 3, 2026

Copy link
Copy Markdown
Member

Summary

  • When image/images are present, open the Gtk MCQ large (~70%+ of the largest monitor) instead of a ~320px-stuck preview.
  • Add header maximize / F, and click-preview to toggle large ↔ compact (~320px). Text-only MCQs stay compact.
  • Document in docs/AGENTS.md + skill (mcq-with-image).

Hosts / platforms touched

  • Cursor (Linux Gtk)
  • Claude Code
  • Claude Desktop / other stdio host
  • Windows Phase 1 (tkinter text-only)
  • Docs only (no behaviour change)

Test plan

CI must stay green (test, secrets-hygiene). Also say what you ran:

Check Result Notes
uv run python scripts/test_match_transcript.py Skip
uv run python scripts/test_doctor.py Skip
uv run python scripts/test_acks.py Skip
uv run python scripts/test_install.py Skip
uv run python scripts/test_dialog_ergonomics.py Skip
uv run python scripts/test_windows_backend.py Skip
uv run python scripts/test_contracts.py Pass
uv run python scripts/test_mcq_images.py Pass
check_setup in a real host Skip
ask_multiple_choice dialog click Pass Direct ask_zenity from worktree
Image preview in Gtk dialog (image/images) Pass 2400×1800 beauty still; Alex: “Feels right”
Keyboard 1–8 / Enter / Esc Skip
Freeform / Something else Skip
Speak / duck (if claiming voice) Skip text-only prefs
Maximize / click-preview Pass Alex confirmed UX

Environment: Ubuntu Linux, Cursor host, local DISPLAY=:0, worktree /data_drive/dd/ask-question-mcp-mcq-image-max, largest monitor 3840×2160.

Behaviour contract

  • No removal/rename of MCP tools without a migration note (ask_multiple_choice, check_setup, setup_guide, record_platform_feedback)
  • Cursor absolute-uv docs kept (GUI thin PATH)
  • No lab IPs / home paths / secrets in tracked files
  • If adding a host: update Tested platforms table (Verified vs Not yet reported must match your results)

AI assistance

  • Human-authored
  • AI-assisted (Cursor) — unit tests + interactive image maximize smoke; Alex confirmed UX and approved this PR create

Made with Cursor

Image MCQs open ~70%+ of the largest monitor, with click-to-toggle
compact/large preview and a header/F maximize control, so stills are
inspectable without crushing text-only dialogs.

Co-authored-by: Cursor <cursoragent@cursor.com>
@ajlennon
ajlennon requested a review from jackghx as a code owner August 3, 2026 00:52
@ajlennon
ajlennon merged commit a701719 into main Aug 3, 2026
4 checks passed
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