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

Use xdg-open to open the native file manager on Linux #284

Closed
jhasse opened this Issue Jan 3, 2018 · 15 comments

Comments

Projects
None yet
2 participants
@jhasse

jhasse commented Jan 3, 2018

I've just tried fman on Ubuntu MATE 17.10 and pressing F10 won't open my native file manager but tell me to configure it. It would be great if this wasn't required and fman would use xdg-open {curr_dir} instead, as that should work for most (if not all) desktop environments.

@mherrmann

This comment has been minimized.

Contributor

mherrmann commented Jan 3, 2018

Interesting, I did not know this. I just implemented your suggestion. It will be released with the next version of fman (usually in a couple of days). You will be notified of the new version automatically.

@mherrmann

This comment has been minimized.

Contributor

mherrmann commented Jan 8, 2018

Implemented in fman 0.8.3. :)

@mherrmann mherrmann closed this Jan 8, 2018

@mherrmann mherrmann removed the in progress label Jan 8, 2018

@jhasse

This comment has been minimized.

jhasse commented Jan 8, 2018

Thanks for the fix :)

Unfortunately it doesn't seem to work for me (just updated fman via apt) - same Popen error message as before. Do I need to reset the settings or something first?

@mherrmann

This comment has been minimized.

Contributor

mherrmann commented Jan 8, 2018

When you initially encountered the problem, did you fix it by editing the settings?

@jhasse

This comment has been minimized.

jhasse commented Jan 8, 2018

No (where are the settings btw? Typing "setting" in the command pallet yields no results).

@mherrmann

This comment has been minimized.

Contributor

mherrmann commented Jan 8, 2018

Here is the revant page of fman's documentation: https://fman.io/docs/terminal. The file you want to look at is ~/.config/fman/Plugins/User/Settings/Core Settings (Linux).json. Does it contain an entry for native_file_manager?

@mherrmann

This comment has been minimized.

Contributor

mherrmann commented Jan 8, 2018

Re-opening this issue until it's confirmed fixed.

@mherrmann mherrmann reopened this Jan 8, 2018

@jhasse

This comment has been minimized.

jhasse commented Jan 8, 2018

The file doesn't exist:

$ tree ~/.config/fman
/home/jhasse/.config/fman
├── Local
│   ├── Cache
│   │   └── Icons
│   ├── Metrics.json
│   └── Session.json
└── Plugins
    └── User
        └── Settings
            ├── Panes (Linux).json
            └── Visited Paths (Linux).json

6 directories, 4 files
@mherrmann

This comment has been minimized.

Contributor

mherrmann commented Jan 8, 2018

Then you have the default settings and for some reason the way fman launches xdg-open does not work. Are you still getting the Please configure error?

@mherrmann

This comment has been minimized.

Contributor

mherrmann commented Jan 8, 2018

(Sorry it's not working btw)

@jhasse

This comment has been minimized.

jhasse commented Jan 8, 2018

Are you still getting the Please configure error?

Yes, exactly like before with 0.8.2.

(Sorry it's not working btw)

No problem ;) Must be quite some work for you to support Linux in the first place.

@mherrmann

This comment has been minimized.

Contributor

mherrmann commented Jan 8, 2018

Ah, I think I know why. Let me fix this for the next version of fman.

@mherrmann

This comment has been minimized.

Contributor

mherrmann commented Jan 23, 2018

fman 0.8.6 is out. Can you let me know whether it fixes the issue for you @jhasse ?

@jhasse

This comment has been minimized.

jhasse commented Jan 24, 2018

Yes, works perfectly! Thank you very much :)

@jhasse jhasse closed this Jan 24, 2018

@mherrmann

This comment has been minimized.

Contributor

mherrmann commented Jan 24, 2018

Cool! Thx for letting me know :-)

@mherrmann mherrmann removed the in progress label Jan 24, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment