Skip to content

Conversation

@mgiovani
Copy link

Summary

Implements the feature requested in #51 to hide the .md extension from all file displays throughout the Haptic application.

Changes Made

  • ✅ Added formatDisplayName() utility function to strip .md extensions
  • ✅ Updated all file display components (sidebar, toolbar, search, command menu)
  • ✅ Applied changes consistently across both web and desktop applications
  • ✅ Maintained backwards compatibility with all file operations

Implementation Details

  • DRY principle: Single reusable utility function used throughout
  • Cross-platform: Works in both web and desktop apps
  • Clean code: Self-documenting without unnecessary comments
  • Zero breaking changes: File operations unchanged

Testing

  • ✅ All linting checks pass
  • ✅ Build compilation successful
  • ✅ No test updates needed (no existing tests)

Fixes #51

- Add formatDisplayName utility function to strip .md extensions
- Update all file display components (sidebar, toolbar, search, etc.)
- Apply changes to both web and desktop applications
- Maintain backwards compatibility with file operations

Fixes chroxify#51
@vercel
Copy link

vercel bot commented Sep 23, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
haptic-web Ready Ready Preview Comment Sep 23, 2025 8:56pm

@vercel
Copy link

vercel bot commented Sep 23, 2025

Someone is attempting to deploy a commit to the chroxify's team Team on Vercel.

A member of the Team first needs to authorize it.

- Add formatFolderPath, formatActiveFileName, and formatFileNameFromPath utility functions
- Replace repetitive folder.name.slice(1).replaceAll('/', ' > ') patterns in command menu
- Replace complex activeFile?.replace($collection, '').split('/').slice(-1)[0] pattern in toolbar
- Replace formatDisplayName(path.split('/').pop()) patterns in search and task components
- Improve code maintainability following DRY principles

Addresses code repetition identified in issue chroxify#51 implementation
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.

Hide .md extension

1 participant