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

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.

kugiigi added 2 commits April 19, 2021 15:04
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