Skip to content

Feat/add configurable keybindings#15

Merged
aikhe merged 4 commits intoaikhe:masterfrom
jara505:feat/add-configurable-keybindings
Mar 20, 2026
Merged

Feat/add configurable keybindings#15
aikhe merged 4 commits intoaikhe:masterfrom
jara505:feat/add-configurable-keybindings

Conversation

@jara505
Copy link
Contributor

@jara505 jara505 commented Mar 16, 2026

Overview

Add configurable keybindings support to allow users to customize dashboard shortcuts. This addresses the need for Neovim users who prefer to define their own keybindings instead of using defaults.

Key Changes

  • Add keys configuration option in state.lua with defaults
  • Update init.lua to handle custom keybindings in setup()
  • Modify ui/init.lua to use configurable keys for close/prev_year/next_year actions
  • Add default keybinding in plugin/wrapped.lua for out-of-box functionality
  • Update README with complete configuration example

Impact

  • plugin/wrapped.lua - Default keybinding
  • lua/wrapped/state.lua - Keys configuration defaults
  • lua/wrapped/init.lua - Setup logic for custom keys
  • lua/wrapped/ui/init.lua - UI keybindings
  • README.md - Documentation

jara505 added 2 commits March 15, 2026 17:41
Add keys configuration option to allow users to customize
dashboard keybindings (open, close, refresh, prev/next year).
@aikhe aikhe self-assigned this Mar 16, 2026
@aikhe
Copy link
Owner

aikhe commented Mar 17, 2026

love this pr, i was too lazy to add it myself lol

merging this once #14 is good

#14 (comment)

@jara505
Copy link
Contributor Author

jara505 commented Mar 19, 2026

Removed the default gw keybinding and updated the README to no longer reference it.
Also added validation for unknown keys in the config, showing messages like: [wrapped] Unknown key: open. Valid keys: close, refresh, prev_year, next_year. @aikhe

@aikhe
Copy link
Owner

aikhe commented Mar 20, 2026

thanks!

@aikhe aikhe merged commit d475a93 into aikhe:master Mar 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants