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

feature: limit columns count to 1 in the helix mode #665

Closed
1 task done
fpohtmeh opened this issue Jul 13, 2024 · 1 comment · Fixed by #663
Closed
1 task done

feature: limit columns count to 1 in the helix mode #665

fpohtmeh opened this issue Jul 13, 2024 · 1 comment · Fixed by #663
Labels
enhancement New feature or request

Comments

@fpohtmeh
Copy link

Did you check the docs?

  • I have read all the which-key.nvim docs

Is your feature request related to a problem? Please describe.

The newly released which-key has a helix preset! But for some mappings, it displays two columns instead of one.
The original Helix editor never displays 2 columns.

image

Describe the solution you'd like

Which key window always has a single-column

Describe alternatives you've considered

Add some option to limit column count

Additional context

Nvim 0.10.0
LazyVim 12.28.0
Which-key 05bd870

@fpohtmeh fpohtmeh added the enhancement New feature or request label Jul 13, 2024
@folke folke closed this as completed in b2a6910 Jul 13, 2024
@folke
Copy link
Owner

folke commented Jul 13, 2024

You could've changed this yourself with opts.layout.width, but it makes sense to add this to the helix preset by default

folke pushed a commit that referenced this issue Jul 14, 2024
🤖 I have created a release *beep* *boop*
---


##
[3.3.0](v3.2.0...v3.3.0)
(2024-07-14)


### Features

* **expand:** allow expand to be a function. Closes
[#670](#670)
([dfaa10c](dfaa10c))
* **mappings:** mapping `desc` and `icon` can now be a function that is
evaluated when which-key is show. Fixes
[#666](#666)
([c634af1](c634af1))
* **mappings:** opts.filter to exclude certain mappings from showing up
in which-key.
([763ea00](763ea00))
* **view:** add operator to trail in op mode
([5a6eaaa](5a6eaaa))
* **view:** when in visual mode, propagate esc. See
[#656](#656)
([30ef44a](30ef44a))


### Bug Fixes

* default preset
([38987d3](38987d3))
* **mappings:** don't show `<SNR>` mappings
([d700244](d700244))
* **presets:** max 1 column in helix mode. Fixes
[#665](#665)
([b2a6910](b2a6910))
* **presets:** shorter descriptions
([9a73d6a](9a73d6a))
* **state:** cooldown till next tick when not safe to open which-key.
Fixes [#672](#672)
([bdf3b27](bdf3b27))
* **util:** nt mode should map to n
([969afc9](969afc9))
* **view:** set nowrap for the which-key window
([6e1c098](6e1c098))
* **view:** set winhl groups. Fixes
[#661](#661)
([baff8ea](baff8ea))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants