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.

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

[1.18] panel can't load launchers for files/folders [$25] #590

Closed
monsta opened this issue Jun 7, 2017 · 12 comments
Closed

[1.18] panel can't load launchers for files/folders [$25] #590

monsta opened this issue Jun 7, 2017 · 12 comments

Comments

@monsta
Copy link
Contributor

monsta commented Jun 7, 2017

@dnk: we have another regression from 7ee4507...

Steps to reproduce:

  1. Launch Caja.
  2. Drag some folder to the panel. I used a folder called "mate".
  3. Launcher for that folder will be created, but won't be loaded. In ~/.xsession-errors there will be an error message: "Unable to open desktop file mate.desktop for panel launcher".

The launcher is stored in ~/.config/mate/panel2.d/default/launchers dir, but for some reason g_desktop_app_info_new_from_filename function (called from create_launcher) can't load it. I don't know the exact reason because that function doesn't have a way to set GError.

Here's the contents of that launcher:

$ cat ~/.config/mate/panel2.d/default/launchers/mate.desktop
#!/usr/bin/env xdg-open
[Desktop Entry]
Version=1.0
Name=mate
Comment=Открыть «/home/monsta/mate»
Icon=folder
Name[ru_RU]=mate
Comment[ru_RU]=Открыть «/home/monsta/mate»
Icon[ru_RU]=folder
URL=file:///home/monsta/mate
Type=Link

Did you help close this issue? Go claim the $25 bounty on Bountysource.

@lukefromdc
Copy link
Member

Confirmed here on Debian Unstable with GTK 3.22.15

@AshleyYakeley
Copy link

Ubuntu Launchpad downstream bug: https://bugs.launchpad.net/ubuntu/+source/mate-panel/+bug/1690517

@raveit65
Copy link
Member

raveit65 commented Aug 4, 2017

It seems to be very popular to create launchers from folders in the panel.
I can confirm that it doesn't work to drag a folder from caja window, but it works to drag the home folder from the desktop for some reasons here.

@raveit65
Copy link
Member

raveit65 commented Aug 4, 2017

@dnk
Can you please take a look in it?

@AshleyYakeley
Copy link

I downgraded to 1.16 to avoid this.

@flexiondotorg flexiondotorg changed the title [1.18] panel can't load launchers for files/folders [1.18] panel can't load launchers for files/folders [$10] Aug 12, 2017
@flexiondotorg flexiondotorg changed the title [1.18] panel can't load launchers for files/folders [$10] [1.18] panel can't load launchers for files/folders [$25] Sep 3, 2017
@monsta
Copy link
Contributor Author

monsta commented Sep 27, 2017

I can confirm that it doesn't work to drag a folder from caja window, but it works to drag the home folder from the desktop for some reasons here.

I know why it works. It creates an app launcher with caja --no-desktop, not a folder link launcher. It's also saved only in dconf like all the other app launchers, nothing is saved in the config directory.

@mexsudo
Copy link

mexsudo commented Oct 3, 2017

Good morning.
Using LMDE. I have been trying daily. Updated again, two different computers. Not fixed. Played with various repos, nothing new to update with.
Is this an issue with Mint not adopting the fix?
thanks

@lukefromdc
Copy link
Member

My guess is Mint won't pick up the fix until the next 1.18 point release.

@raveit65
Copy link
Member

raveit65 commented Oct 3, 2017

Is this an issue with Mint not adopting the fix?

This you have to ask linuxmint devs.
We are MATE, not linuxmint ;-)

@monsta
Copy link
Contributor Author

monsta commented Nov 12, 2017

Fix is now released in version 1.18.5.

@raveit65
Copy link
Member

@monsta
Copy link
Contributor Author

monsta commented Sep 17, 2018

I didn't even notice there was a bounty... Ok, I submitted a claim there.

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

Successfully merging a pull request may close this issue.

6 participants