Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat: add open directory action #132

Closed
wants to merge 1 commit into from
Closed

feat: add open directory action #132

wants to merge 1 commit into from

Conversation

ArnoChenFx
Copy link

@ArnoChenFx ArnoChenFx commented Jan 25, 2025 •
edited
Loading

Description

This PR adds the ability to open a process's executable directory directly from the UI. The implementation includes:

  • New Tauri command open_process_directory handling cross-platform path opening
  • System monitor integration to retrieve process executable paths
  • Frontend UI components with a new folder open button (use faFolderOpen icon)
  • Process store updates to handle directory opening operations
  • Windows/macOS/Linux platform-specific file explorer support

image

Type of change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

  • Windows - Verify that explorer correctly opens the process directory
  • macOS - Verify the behavior of the open command
  • Linux - Verify the behavior of xdg-open
  • Test handling of non-existent process paths
  • Verify UI button click events and state updates

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • New and existing unit tests pass locally with my changes

@ArnoChenFx ArnoChenFx force-pushed the add-open-directory-action branch from d045834 to 4940d30 Compare January 25, 2025 14:44
@ArnoChenFx ArnoChenFx force-pushed the add-open-directory-action branch from 4940d30 to c7d8726 Compare January 27, 2025 13:22
@Abdenasser
Copy link
Owner

thank you, will take a look to this asap.

@Abdenasser Abdenasser closed this Feb 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Abdenasser Abdenasser Awaiting requested review from Abdenasser Abdenasser is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ArnoChenFx @Abdenasser