Skip to content

fix(penpal): show agent selector dropdown below trigger button - #532

Merged
matt2e merged 1 commit into
mainfrom
project-agent-drop-down
Mar 29, 2026
Merged

fix(penpal): show agent selector dropdown below trigger button#532
matt2e merged 1 commit into
mainfrom
project-agent-drop-down

Conversation

@matt2e

@matt2e matt2e commented Mar 29, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Fix agent selector dropdown positioning to appear below the trigger button instead of above it
  • Change CSS from bottom: 100% to top: 100% and margin-bottom to margin-top for correct placement

Test plan

  • Open the agent selector in the penpal UI
  • Verify the dropdown appears below the trigger button
  • Verify proper spacing between button and dropdown

🤖 Generated with Claude Code

The dropdown was positioned above the button using `bottom: 100%`,
causing it to overflow above the top of the page. Changed to `top: 100%`
so it appears below the trigger where it's fully visible.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@matt2e
matt2e requested review from baxen and wesbillman as code owners March 29, 2026 23:44
@matt2e
matt2e merged commit cd09878 into main Mar 29, 2026
4 checks passed
@matt2e
matt2e deleted the project-agent-drop-down branch March 29, 2026 23:53
loganj added a commit that referenced this pull request Mar 30, 2026
…-file-dis

* origin/main:
  fix(penpal): ensure repo badges are generated when repos are added (#534)
  fix(penpal): show agent selector dropdown below trigger button (#532)
  fix(penpal): filter ANCHORS.md from anchors source (#533)
  fix(penpal): show wave animation in sidebar when run phase is unknown (#531)
  fix(penpal): anchor stability improvements and code block highlights (#530)
  refactor(penpal): decompose Layout.tsx into focused components (#528)

# Conflicts:
#	apps/penpal/frontend/src/components/Layout.tsx
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