-
-
Notifications
You must be signed in to change notification settings - Fork 6
Search #49
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Search #49
Conversation
…arch within directory features added ### 1. Enhanced Keyboard Shortcuts - **Ctrl+H** - Toggle hidden files (in addition to existing '.') - **Ctrl+L** - Focus path input field for direct path entry - **Ctrl+R** - Toggle recent locations panel - **Ctrl+F** - Toggle search mode and focus search input - **F5** - Refresh current directory - **Ctrl+D** - Bookmark current directory (placeholder for future implementation) ### 2. Recent Locations Panel - Shows recently accessed files and directories - Toggle visibility with Ctrl+R - Click to quickly navigate to recent locations - Placeholder for persistent storage (needs implementation) ### 3. Breadcrumb Navigation - Visual path display with clickable components - Click any path segment to navigate directly - Better understanding of current location in filesystem hierarchy ### 4. Search Within Directory - Real-time search filtering of directory contents - Toggle with Ctrl+F - Filters files and folders by name - Clear button to reset search ### 5. Improved Visual Feedback - Notifications for keyboard actions - Better error messages - Search active indicator
Remove extra commetns
I recognize and realize this is probably not 'ideal' code, and do not mind if you reject/rewrite it all. Just wanting to contribute what I've added to your library to help me, back into main and all that. |
Thanks, I appreciate the honesty here. Sadly, I'm not sure I can in good conscience accept code created in this way, especially given the possible licence implications. Moreover, a quick glance over the code throws up quite a few problems with it. Probably the most glaring issue is that this PR renders the library impossible to import because of one particular change it makes.
|
Sorry about the other pull request, here's a slightly better one,
Full Disclosure: This code was vibe coded via Anthropic Opus (claude code) -> Reviewed -> Implemented -> Iterated -> Tested -> Worked.