Page MenuHomeLubuntu Development

Edited shortcuts so that they don't interfere with openbox key bindings.

Authored by The_LoudSpeaker on Jul 1 2019, 11:32 PM.


Maniphest Tasks
T59: fix keyboard shortcuts
  • Edited the foloowing shortcuts:
    • GoToDesktop
    • Show Desktop
    • pcmanfm-qt
    • runner
    • qps
    • terminal
Test Plan

Apply this along with D34 and Check all things mentioned in T59#1260 are satisfied.

Diff Detail

rLXQTGLOBALKEYSPACKAGING lxqt-globalkeys Packaging
Lint Skipped
Unit Tests Skipped

Event Timeline

The_LoudSpeaker created this revision.
tsimonq2 requested changes to this revision.Jul 2 2019, 1:37 AM

Everything under debian/lxqt-globalkeys-l10n needs to be removed.

This revision now requires changes to proceed.Jul 2 2019, 1:37 AM

Yeah and let's slow our roll here. We might want to see if lxqt-globalkeys doesn't actually support XDG. It should. If so, this can go in default settings after all.

One thing you could do while we figure this out, though, is to clean up your diff as aforementioned.

So I've confirmed it is hardcoded and it seems to be a known problem but it's kind of a bugaboo. @tsimonq2 maybe we should just go the route of patching globalkeys now and later when we get the patch to respect XDG, we can move that patch to default-settings?

Removed debian/lxqt-globalkeys-l10n

Regarding the patch, remove this:

Origin: <upstream|backport|vendor|other>, <URL, required except if Author is present>
Bug: <URL to the upstream bug report if any, implies patch has been forwarded, optional>
Bug-<Vendor>: <URL to the vendor bug report if any, optional>
Forwarded: <URL|no|not-needed, useless if you have a Bug field, optional>
Applied-Upstream: <version|URL|commit, identifies patches merged upstream, optional>
Reviewed-by: <name and email of a reviewer, optional>
This patch header follows DEP-3:

Your date has some extra cruft, too, and should just be:

Last-Update: 2019-07-02

I'd recommend adding:


Finally, I think we need to make a decision about whether or not we *are* patching this again here or whether or not we should wait for full XDG support upstream. @tsimonq2 hello???

wxl requested changes to this revision.Jul 7 2019, 3:13 AM
This revision now requires changes to proceed.Jul 7 2019, 3:13 AM

Added link to the bug in patch description

wxl requested changes to this revision.Jul 7 2019, 3:42 AM

Sorry to be particular, but we're training you, so we're going to make you do it right before we set you free to fly on your own ๐Ÿ˜„ That said, please remove this:

This patch header follows DEP-3:
This revision now requires changes to proceed.Jul 7 2019, 3:42 AM

updated the patch description

This comment was removed by wxl.
wxl requested changes to this revision.EditedJul 8 2019, 4:48 AM

path=/panel/desktopswitch2 should be path=/panel/desktopswitch

One other thing that should be added here (and should be removed from Openbox) is all the media keys (the XF86 buttons). You can just edit globalkeyshortcuts.conf.

This revision now requires changes to proceed.Jul 8 2019, 4:48 AM

Looks like there's a potential upstream pull request for making globalkeys respect XDG, so this may end up going to rDEFAULTSETTINGS after all.

We test to see if it works. If it does, then we will need to prioritize that first. I doubt we will get a new release, so we'll have to patch our version in Ubuntu. Once that's settled, then we deal with putting this in the right place. According to the Qt docs, barring a local config, QSettings (which is what the pull request uses) will look through XDG_CONFIG_DIRS. As our wiki implies, this should equal /etc/xdg/xdg-Lubuntu:/etc/xdg:/etc:/usr/share. rDEFAULTSETTINGS already has a /etc/xdg/xdg-Lubuntu/lxqt folder so that's where this would end up if this does work. Naturally, what we'll be putting in there is the whole file and not this patch, so things will be a bit different in final implementation.

The patch got merged, so someone needs to get that in if we don't get a new release. @tsimonq2 do you know if they plan on doing an LXQt release before, say, FeatureFreeze?

wxl requested changes to this revision.Aug 21 2019, 4:48 AM

A few problems:

  1. In that last change, you killed most of the changelog.
  2. Don't you want to use your email?
  3. You need to update the changelog date. dch -r "" should do the trick.
This revision now requires changes to proceed.Aug 21 2019, 4:48 AM

The good news: we've got the lxqt-globalkeys patch in proposed.

Since it's done, we don't have to then put the config file here right? It should go to rLUBUNTUDEFAULTSETTINGS ?