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(fzf-lua): update update_title and update_scrollbar to update_preview_title and update_preview_scrollbar #1034

Merged
merged 1 commit into from
Feb 6, 2025

Conversation

zhiim
Copy link
Contributor

@zhiim zhiim commented Jan 10, 2025

Description

FzfWin:update_title and FzfWin:update_scrollbar were changed to FzfWin:update_preview_title and FzfWin:update_preview_scroolbar since ibhagwan/fzf-lua@fc70792

It will make an error when opening message history with fzf-lua :Noice fzf

Error executing vim.schedule lua callback: ...hare/nvim/lazy/noice.nvim/lua/noice/integrations/fzf.lua:75: attempt to call method 'update__title' (a nil value)
stack traceback:
	...hare/nvim/lazy/noice.nvim/lua/noice/integrations/fzf.lua:75: in function 'populate_preview_buf'
	...hare/nvim/lazy/fzf-lua/lua/fzf-lua/previewer/builtin.lua:377: in function 'populate_preview_buf'
	...hare/nvim/lazy/fzf-lua/lua/fzf-lua/previewer/builtin.lua:394: in function ''
	vim/_editor.lua: in function ''
	vim/_editor.lua: in function <vim/_editor.lua:0>

Related Issue(s)

Fixes #1029

@aaronlifton
Copy link
Contributor

Did this locally as well. This is needed for the fzf lua integration to work.

@folke folke merged commit 5530d78 into folke:main Feb 6, 2025
2 checks passed
@folke
Copy link
Owner

folke commented Feb 6, 2025

ty

folke pushed a commit that referenced this pull request Feb 6, 2025
## Description
`<leader>snt` wasn't working properly

- the same fzf compatibility update as
#1034
- the hl_group update from #1032
but for `fzf.lua`

Those PRs should probably merged instead of this, but the hl_group
update one also needs to update `lua/noice/integrations/fzf.lua`
Figured I'd submit this to bump visibility of the issue.

## Related Issue(s)

#1029
#1024
folke pushed a commit that referenced this pull request Feb 6, 2025
🤖 I have created a release *beep* *boop*
---


##
[4.10.0](v4.9.0...v4.10.0)
(2025-02-06)


### Features

* **fzf:** fix fzf integration
([#1048](#1048))
([f3291db](f3291db))


### Bug Fixes

* **cmdline:** use number indexed `vim.g.ui_cmdline_pos`
([eaed6cc](eaed6cc))
* **confirm:** fixed `vim.fn.confirm` on nightly (0.11). Closes
[#1036](#1036)
([00b5984](00b5984))
* **fzf-lua:** update `update_title` and `update_scrollbar` to
`update_preview_title` and `update_preview_scrollbar`
([#1034](#1034))
([5530d78](5530d78)),
closes [#1029](#1029)
* **nui:** don't error when border win was closed. Fixes
[#832](#832)
([d2b092a](d2b092a))
* **telescope:** handle hl_group ID correctly in picker
([#1032](#1032))
([002b202](002b202)),
closes [#1024](#1024)

---
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
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug: :Noice fzf error
3 participants