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

fix: Fix invalid order of event handling in input/text fields #284

Merged
merged 2 commits into from
Jul 22, 2024

Conversation

Sculas
Copy link
Contributor

@Sculas Sculas commented Jun 18, 2024

Follow up of charmbracelet/bubbletea#1041, although they do not directly require each other.

Key binds should be checked before the key event is accepted as user input. Otherwise, control sequences or other keys might be printed instead of being interpreted as a key bind. See the above-mentioned PR for more information.

Key binds should be checked before the key event is accepted as user input. See charmbracelet/bubbletea#1041 for more information.
@maaslalani maaslalani merged commit dad7190 into charmbracelet:main Jul 22, 2024
20 checks passed
@maaslalani
Copy link
Contributor

Thanks so much for the contribution!!

@Sculas Sculas deleted the fix/invalid-event-order branch July 22, 2024 17:32
renovate bot referenced this pull request in jippi/dottie Jul 25, 2024
….mod (#67)

[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [github.com/charmbracelet/huh](https://togithub.com/charmbracelet/huh)
| `v0.5.1` -> `v0.5.2` |
[![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fcharmbracelet%2fhuh/v0.5.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fcharmbracelet%2fhuh/v0.5.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fcharmbracelet%2fhuh/v0.5.1/v0.5.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fcharmbracelet%2fhuh/v0.5.1/v0.5.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>charmbracelet/huh (github.com/charmbracelet/huh)</summary>

###
[`v0.5.2`](https://togithub.com/charmbracelet/huh/releases/tag/v0.5.2)

[Compare
Source](https://togithub.com/charmbracelet/huh/compare/v0.5.1...v0.5.2)

### Lil’ fixes ’n’ improvements

Hi! This is a maintenance release to fix issues with dynamic forms as
well as address issues with
[Gum](https://togithub.com/charmbracelet/gum) upstream.

#### Changelog

##### New

- add accept and reject key bindings to confirm by
[@&#8203;csandeep](https://togithub.com/csandeep) in
[https://github.com/charmbracelet/huh/pull/308](https://togithub.com/charmbracelet/huh/pull/308)
- allow Generic `T` updateOptionsMsg by
[@&#8203;maaslalani](https://togithub.com/maaslalani) in
[https://github.com/charmbracelet/huh/pull/318](https://togithub.com/charmbracelet/huh/pull/318)

##### Fixed

- improve distinction in field select by
[@&#8203;csandeep](https://togithub.com/csandeep) in
[https://github.com/charmbracelet/huh/pull/304](https://togithub.com/charmbracelet/huh/pull/304)
- fix invalid order of event handling in input/text fields by
[@&#8203;Sculas](https://togithub.com/Sculas) in
[https://github.com/charmbracelet/huh/pull/284](https://togithub.com/charmbracelet/huh/pull/284)

#### New Contributors

- [@&#8203;csandeep](https://togithub.com/csandeep) made their first
contribution in
[https://github.com/charmbracelet/huh/pull/304](https://togithub.com/charmbracelet/huh/pull/304)
-
[@&#8203;MaximilianSoerenPollak](https://togithub.com/MaximilianSoerenPollak)
made their first contribution in
[https://github.com/charmbracelet/huh/pull/313](https://togithub.com/charmbracelet/huh/pull/313)

**Full Changelog**:
charmbracelet/huh@v0.5.1...v0.5.2

***

<a href="https://charm.sh/"><img alt="The Charm logo"
src="https://stuff.charm.sh/charm-badge.jpg" width="400"></a>

Thoughts? Questions? We love hearing from you. Feel free to reach out on
[Twitter](https://twitter.com/charmcli), [The
Fediverse](https://mastodon.technology/@&#8203;charm), or
[Discord](https://charm.sh/chat).

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "* */8 * * *" (UTC), Automerge - At
any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR was generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View the
[repository job log](https://developer.mend.io/github/jippi/dottie).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy40MzguMCIsInVwZGF0ZWRJblZlciI6IjM3LjQzOC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJkZXBlbmRlbmNpZXMiXX0=-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
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