Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add alt + shift modifiers #316

Merged
merged 14 commits into from
Jun 14, 2023

Conversation

elmarsan
Copy link
Contributor

This pull request aims to add support for Alt and Shift modifiers.

Closes #33

@maaslalani
Copy link
Member

maaslalani commented Jun 9, 2023

Hey @elmarsan, this PR looks great.

1 suggestion is can we change the syntax (in the docs) from:

Ctrl+N<char|modifier>

to:

Ctrl[+Alt][+Shift]+<char>

parser.go Outdated Show resolved Hide resolved
elmarsan and others added 12 commits June 14, 2023 14:55
Co-authored-by: Maas Lalani <maas@lalani.dev>
* feat: setting to control cursor blinking

* feat: create bool type

* feat: parser updated

* readme wording

Co-authored-by: Maas Lalani <maas@lalani.dev>

* Update README.md

Co-authored-by: Maas Lalani <maas@lalani.dev>

* Update parser.go

* Update token.go

* Update parser.go

* Apply suggestions from code review

---------

Co-authored-by: Maas Lalani <maas@lalani.dev>
…bracelet#321)

Bumps [github.com/go-rod/rod](https://github.com/go-rod/rod) from 0.112.8 to 0.113.3.
- [Release notes](https://github.com/go-rod/rod/releases)
- [Commits](go-rod/rod@v0.112.8...v0.113.3)

---
updated-dependencies:
- dependency-name: github.com/go-rod/rod
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…armbracelet#310)

Bumps [github.com/mattn/go-isatty](https://github.com/mattn/go-isatty) from 0.0.18 to 0.0.19.
- [Commits](mattn/go-isatty@v0.0.18...v0.0.19)

---
updated-dependencies:
- dependency-name: github.com/mattn/go-isatty
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Added note to the comment that is generated when a user calls `vhs new` to call out that `TypingSpeed` can be customized globally, and defaults to 50ms
@elmarsan
Copy link
Contributor Author

@maaslalani it should be ready 😀

@maaslalani
Copy link
Member

Awesome @elmarsan, going to get this one in today for sure after testing it out.

@maaslalani maaslalani merged commit 4dd9a81 into charmbracelet:main Jun 14, 2023
7 checks passed
@maaslalani
Copy link
Member

Amazing work @elmarsan! This is fantastic.

@maaslalani maaslalani deleted the feat/add-alt-shift-modifiers branch June 14, 2023 14:34
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.

Alt + Shift modifiers
6 participants