Skip to content
This repository has been archived by the owner on Jun 28, 2024. It is now read-only.

feat: add keyboard shortcuts (#776) #777

Merged
merged 3 commits into from
May 28, 2023
Merged

feat: add keyboard shortcuts (#776) #777

merged 3 commits into from
May 28, 2023

Conversation

0xMRTT
Copy link
Member

@0xMRTT 0xMRTT commented May 26, 2023

Description

image

Fixes #776

Type of change

  • Bugfix (Change which fixes a issue)
  • New feature (Change which adds new functionality)
  • Enhancement (Change which slightly improves existing code)
  • Breaking change (This change will introduce incompatibility with existing functionality)

Changelog

  • Added Help Overlay

Testing

  • I have tested my changes and verified that they work as expected

@0xMRTT 0xMRTT requested review from tfuxu and daudix May 26, 2023 22:48
daudix
daudix approved these changes May 26, 2023
View reviewed changes
Copy link
Member

@daudix daudix left a comment

Choose a reason for hiding this comment

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

Something that were missing and caused this feeling of that something is off, LGTM!

@0xMRTT
Copy link
Member Author

0xMRTT commented May 26, 2023

image

@0xMRTT
Copy link
Member Author

0xMRTT commented May 26, 2023

Now it should be Okay!

tfuxu
tfuxu approved these changes May 28, 2023
View reviewed changes
Copy link
Member

@tfuxu tfuxu left a comment

Choose a reason for hiding this comment

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

Great little addition to Gradience. LGTM

@tfuxu tfuxu merged commit 2f9f879 into main May 28, 2023
5 checks passed
@tfuxu tfuxu deleted the shortcuts branch May 28, 2023 21:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@tfuxu tfuxu tfuxu approved these changes

@daudix daudix daudix approved these changes

Assignees
No one assigned
Labels
enhancement New feature or request status/pr-approved
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

feat: Add shortcuts
3 participants
@0xMRTT @tfuxu @daudix