Skip to content
This repository has been archived by the owner on Nov 4, 2023. It is now read-only.

Disable shortcuts when launcher is not available #379

Open
wants to merge 2 commits into
base: xenial
Choose a base branch
from

Conversation

kugiigi
Copy link
Contributor

@kugiigi kugiigi commented Apr 19, 2021

This disables keyboard shortcuts when the launcher is disabled or not available.

i.e. Launcher is disabled in lock screen.
Currently, swiping won't show the launcher if it's disabled in lock screen but keyboard shortcuts are still working.

This will disable launcher shortcuts when launcher is disabled or not available i.e. Launcher is disabled in lock screen.
@UniversalSuperBox
Copy link
Member

This PR looks good to me, but I believe a unit test should be added to ensure that the shortcuts do not function when the shell is locked. This will ensure that we don't add this bug back in the future.

Copy link
Member

@dobey dobey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a test to ensure this doesn't break in the future, as Dalton requested.
Also, it might be good to clean up the commits by squashing with git rebase -i after.

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

Successfully merging this pull request may close these issues.

3 participants