search

aperezdc / revolt

Better desktop integration for Riot.im (not only) for GNOME
GNU General Public License v3.0
92 stars 9 forks source link

App menu and keyboard shortcuts disabled #87

Closed philn closed 6 years ago

philn commented 6 years ago

This seems to be an issue with the Flatpak version of Revolt. The app menu is greyed out and keyboard shortcuts don't work at all.

Uninstalled (make run) version works as expected. Other Flatpak apps work as expected, I tried the PiTiVi app.

philn commented 6 years ago

Maybe this is relevant?

Feb 12 09:16:06 ecto-1 systemd[14005]: Started flatpak document portal service.
Feb 12 09:16:06 ecto-1 systemd[14005]: flatpak-com.igalia.Revolt-15876.scope: Failed to add PIDs to scope's control group: Permission denied
Feb 12 09:16:06 ecto-1 systemd[14005]: flatpak-com.igalia.Revolt-15876.scope: Failed with result 'resources'.
Feb 12 09:16:06 ecto-1 systemd[14005]: Failed to start flatpak-com.igalia.Revolt-15876.scope.
aperezdc commented 6 years ago

I have been able of reproducing this issue after upgrading to Flatpak 0.10.3 myself. For the moment I do not have a proper fix for the menu, but as a stopgap measure I have pushed commit https://github.com/aperezdc/revolt/commit/61646b20f5136dd43c8a77a34ef6624acb53e077, which adds options in the contextual menu to open the preferences window — the Flatpak repository is now updated with this change as well. Having the context menu tweaked is something I had been thinking about before, so it's anyway a small improvement which would be done anyway :wink:

aperezdc commented 6 years ago

@philn: Both myself and @jaragunde have the menu entries working fine after commit https://github.com/aperezdc/revolt/commit/61646b20f5136dd43c8a77a34ef6624acb53e077 so if you could please confirm that they are working for you, we can close this issue. Thanks!

philn commented 6 years ago

Yes it works now, good job!

aperezdc commented 6 years ago

@philn: Great, thanks for confirming and closing the issue :+1: