zz85 / space-radar

Disk And Memory Space Visualization App built with Electron & d3.js
https://github.com/zz85/space-radar-electron/releases
1.38k stars 80 forks source link

open a directory #33

Closed vanzandtj closed 6 years ago

vanzandtj commented 7 years ago

I'm using space-radar on a Mac. When I right-click on a directory, the "open" option is grayed out. I suggest that this menu option should work the same way as "open" on a command line - that it should launch Finder on the directory. That's the interface most familiar to a new user. (I suppose the corresponding tool under Windows would be the Windows Explorer).

I would also like the option to open a terminal window in the directory, since for an experienced user there are many actions that are easier from the command line.

zz85 commented 7 years ago

May I know what version you're using? Right click > locate works for me (lastest from git repo).

vanzandtj commented 7 years ago

I don't see a version number. I installed it yesterday with "brew cask install spaceradar", so it's whatever version was last "brewed" or whatever.

Here's the content of info.plist:

$ less Info.plist

<?xml version="1.0" encoding="UTF-8"?>

<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" " http://www.apple.com/DTDs/PropertyList-1.0.dtd">

BuildMachineOSBuild 14F1021 CFBundleDisplayName SpaceRadar CFBundleExecutable Electron CFBundleIconFile atom.icns CFBundleIdentifier com.electron.spaceradar CFBundleInfoDictionaryVersion 6.0 CFBundleName SpaceRadar CFBundlePackageType APPL CFBundleShortVersionString 0.34.3 CFBundleVersion 0.34.3 DTSDKBuild 15A278 DTSDKName macosx10.11 DTXcode 0710 DTXcodeBuild 7B91b LSApplicationCategoryType public.app-category.developer-tools LSMinimumSystemVersion 10.8.0 NSHighResolutionCapable NSMainNibFile MainMenu NSPrincipalClass AtomApplication NSSupportsAutomaticGraphicsSwitching CFBundleURLTypes On Tue, Sep 12, 2017 at 12:49 PM, Joshua Koo wrote: > May I know what version you're using? Right click > locate works for me > (lastest from git repo). > > — > You are receiving this because you authored the thread. > Reply to this email directly, view it on GitHub > , > or mute the thread > > . >
vanzandtj commented 7 years ago

Ah - "locate" works for me too. That's a misleading label, though. It ought to say "open in finder" or just "open".

And, yes, I'm reluctant to click on random links just to see what happens. Especially when the specific label I'm looking for is actually there, but grayed out.

On Tue, Sep 12, 2017 at 12:54 PM, James Van Zandt jim.vanzandt@gmail.com wrote:

I don't see a version number. I installed it yesterday with "brew cask install spaceradar", so it's whatever version was last "brewed" or whatever.

Here's the content of info.plist:

$ less Info.plist

<?xml version="1.0" encoding="UTF-8"?>

<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" " http://www.apple.com/DTDs/PropertyList-1.0.dtd">

BuildMachineOSBuild 14F1021 CFBundleDisplayName SpaceRadar CFBundleExecutable Electron CFBundleIconFile atom.icns CFBundleIdentifier com.electron.spaceradar CFBundleInfoDictionaryVersion 6.0 CFBundleName SpaceRadar CFBundlePackageType APPL CFBundleShortVersionString 0.34.3 CFBundleVersion 0.34.3 DTSDKBuild 15A278 DTSDKName macosx10.11 DTXcode 0710 DTXcodeBuild 7B91b LSApplicationCategoryType public.app-category.developer-tools LSMinimumSystemVersion 10.8.0 NSHighResolutionCapable NSMainNibFile MainMenu NSPrincipalClass AtomApplication NSSupportsAutomaticGraphicsSwitching CFBundleURLTypes On Tue, Sep 12, 2017 at 12:49 PM, Joshua Koo wrote: > May I know what version you're using? Right click > locate works for me > (lastest from git repo). > > — > You are receiving this because you authored the thread. > Reply to this email directly, view it on GitHub > , > or mute the thread > > . >
zz85 commented 6 years ago

This is addressed in #35. I'll be naming it "Open directory" and "Open file" to make things clearer.