Skip to content

Conversation

@ibetitsmike
Copy link
Contributor

PR #637 accidentally removed the DirectoryPickerModal integration added in PR #682.

Changes

ProjectCreateModal.tsx:

  • Restored desktop mode: native pickDirectory() dialog
  • Restored browser mode: web-based DirectoryPickerModal component
  • Browse button conditionally shown based on platform capabilities
  • Theme-aware styling (text-foreground instead of hardcoded text-white)

ProjectCreateModal.stories.tsx (new):

  • 8 Storybook tests covering directory picker flow
  • Tests for navigation, selection, cancel, and validation errors

Generated with mux

PR #637 accidentally removed the DirectoryPickerModal integration that was
added in PR #682. This restores:

- Desktop mode: native pickDirectory() dialog
- Browser mode: web-based DirectoryPickerModal component
- Conditional Browse button based on platform capabilities
- Theme-aware styling (text-foreground instead of hardcoded text-white)

Also adds ProjectCreateModal.stories.tsx with 8 Storybook tests covering:
- Basic modal rendering
- Typing path manually
- Browse button opening directory picker
- Directory navigation
- Path selection flow
- Full end-to-end flow
- Cancel behavior
- Validation errors
@ibetitsmike ibetitsmike added this pull request to the merge queue Nov 26, 2025
Merged via the queue into main with commit 6182ee5 Nov 26, 2025
13 checks passed
@ibetitsmike ibetitsmike deleted the fix-directory-picker-modal branch November 26, 2025 06:00
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