{"action":"closed","issue":{"url":"https://api.github.com/repos/linuxmint/cinnamon-spices-applets/issues/1985","repository_url":"https://api.github.com/repos/linuxmint/cinnamon-spices-applets","labels_url":"https://api.github.com/repos/linuxmint/cinnamon-spices-applets/issues/1985/labels{/name}","comments_url":"https://api.github.com/repos/linuxmint/cinnamon-spices-applets/issues/1985/comments","events_url":"https://api.github.com/repos/linuxmint/cinnamon-spices-applets/issues/1985/events","html_url":"https://github.com/linuxmint/cinnamon-spices-applets/issues/1985","id":364772901,"node_id":"MDU6SXNzdWUzNjQ3NzI5MDE=","number":1985,"title":"batterymonitor@pdcurtis: determining editor is fragile","user":{"login":"spantaleev","id":388669,"node_id":"MDQ6VXNlcjM4ODY2OQ==","avatar_url":"https://avatars3.githubusercontent.com/u/388669?v=4","gravatar_id":"","url":"https://api.github.com/users/spantaleev","html_url":"https://github.com/spantaleev","followers_url":"https://api.github.com/users/spantaleev/followers","following_url":"https://api.github.com/users/spantaleev/following{/other_user}","gists_url":"https://api.github.com/users/spantaleev/gists{/gist_id}","starred_url":"https://api.github.com/users/spantaleev/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/spantaleev/subscriptions","organizations_url":"https://api.github.com/users/spantaleev/orgs","repos_url":"https://api.github.com/users/spantaleev/repos","events_url":"https://api.github.com/users/spantaleev/events{/privacy}","received_events_url":"https://api.github.com/users/spantaleev/received_events","type":"User","site_admin":false},"labels":[{"id":509701660,"node_id":"MDU6TGFiZWw1MDk3MDE2NjA=","url":"https://api.github.com/repos/linuxmint/cinnamon-spices-applets/labels/BUG","name":"BUG","color":"ee0701","default":false},{"id":817894256,"node_id":"MDU6TGFiZWw4MTc4OTQyNTY=","url":"https://api.github.com/repos/linuxmint/cinnamon-spices-applets/labels/In%20Progress","name":"In Progress","color":"ededed","default":false}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2018-09-28T07:29:56Z","updated_at":"2018-11-25T23:51:44Z","closed_at":"2018-11-25T23:51:44Z","author_association":"NONE","body":" * Applet version/Build date: 1.3.2\r\n * Cinnamon version (cinnamon --version): 3.8.9\r\n * Distribution: Archlinux\r\n\r\n**Notify author of applet**\r\n@pdcurtis \r\n\r\n**Issue**\r\n\r\nPressing the \"Open stylesheet (Advanced Function)\" button doesn't work.\r\n\r\nLooking at the code, it seems like it checks the Cinnamon version and decides to use `gedit` (if the version is below 3.0) or `xed` otherwise.\r\n\r\nIn my case (Cinnamon 3.8.9), it would try to use `xed`, which is not available.\r\nAs a workaround, one could do: `ln -s /usr/bin/gedit /usr/local/bin/xed` and fake having `xed`.\r\n\r\nWhat would be better, though, is if the extension didn't rely on version detection, but rather just used `xdg-open`.\r\nI'm not sure if `xdg-open` (`xdg-utils`) is something that the Cinnamon desktop can be sure is available though. It doesn't seem like it's a required dependency for it.\r\n\r\nAlternatively, it could check if the `xdg-open` program is available.. And then fall-back to checking other stuff.\r\nIt seems like the extension checks for things like `sox` and `zenity` using `GLib.find_program_in_path()`, so why not use that to check for `xdg-open`, `gedit`, `xed`, etc."}}
{"actor":{"display_login":"jaszhix"}}