Skip to content

Conversation

atusy
Copy link
Contributor

@atusy atusy commented Nov 22, 2023

current implementation of require("flash").treesitter() automatically selects the smallest node.
However, this may cause unintentional scroll of the buffer like below.

2023-11-22 12-06-17 mkv

This PR adds a functionality to opt-out the current behavior by respecting autojump configuration on require("flash").treesitter().

@atusy atusy force-pushed the treesitter-no-autojump branch from 1ef64cc to 36ff924 Compare November 22, 2023 03:13
@atusy atusy changed the title feat(nvim): opt-out selection of the smallest node with flash.treesitter() by specifying autojump = false feat(treesitter): opt-out selection of the smallest node with flash.treesitter() by specifying autojump = false Nov 22, 2023
@atusy atusy force-pushed the treesitter-no-autojump branch from 36ff924 to e8e707a Compare November 22, 2023 04:39
Copy link
Contributor

github-actions bot commented Jul 6, 2024

This PR is stale because it has been open 60 days with no activity.

@github-actions github-actions bot added the stale label Jul 6, 2024
@folke folke merged commit daa06cd into folke:main Jul 7, 2024
@folke
Copy link
Owner

folke commented Jul 7, 2024

ty!

folke pushed a commit that referenced this pull request Jul 7, 2024
🤖 I have created a release *beep* *boop*
---


## [2.1.0](v2.0.0...v2.1.0)
(2024-07-07)


### Features

* **prompt:** add `flash.prompt()` to get the prompt text
([#347](#347))
([f226e02](f226e02))
* **treesitter:** opt-out selection of the smallest node with
flash.treesitter() by specifying autojump = false
([#276](#276))
([daa06cd](daa06cd))


### Bug Fixes

* **state:** abort to use Util.exit()
([#324](#324))
([e7ab1e4](e7ab1e4))
* **treesitter:** process all trees. Closes
[#275](#275)
([51f3ee1](51f3ee1))

---
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>
myypo pushed a commit to myypo/flash.nvim that referenced this pull request Aug 10, 2024
…reesitter() by specifying autojump = false (folke#276)

current implementation of `require("flash").treesitter()` automatically
selects the smallest node.
However, this may cause unintentional scroll of the buffer like below.

![2023-11-22 12-06-17
mkv](https://github.com/folke/flash.nvim/assets/30277794/11bf5d0c-88be-45f6-bd7f-b8ecfc982756)

This PR adds a functionality to opt-out the current behavior by
respecting `autojump` configuration on `require("flash").treesitter()`.

Co-authored-by: Folke Lemaitre <[email protected]>
myypo pushed a commit to myypo/flash.nvim that referenced this pull request Aug 10, 2024
🤖 I have created a release *beep* *boop*
---


## [2.1.0](folke/flash.nvim@v2.0.0...v2.1.0)
(2024-07-07)


### Features

* **prompt:** add `flash.prompt()` to get the prompt text
([folke#347](folke#347))
([f226e02](folke@f226e02))
* **treesitter:** opt-out selection of the smallest node with
flash.treesitter() by specifying autojump = false
([folke#276](folke#276))
([daa06cd](folke@daa06cd))


### Bug Fixes

* **state:** abort to use Util.exit()
([folke#324](folke#324))
([e7ab1e4](folke@e7ab1e4))
* **treesitter:** process all trees. Closes
[folke#275](folke#275)
([51f3ee1](folke@51f3ee1))

---
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>
myypo pushed a commit to myypo/flash.nvim that referenced this pull request Aug 10, 2024
…reesitter() by specifying autojump = false (folke#276)

current implementation of `require("flash").treesitter()` automatically
selects the smallest node.
However, this may cause unintentional scroll of the buffer like below.

![2023-11-22 12-06-17
mkv](https://github.com/folke/flash.nvim/assets/30277794/11bf5d0c-88be-45f6-bd7f-b8ecfc982756)

This PR adds a functionality to opt-out the current behavior by
respecting `autojump` configuration on `require("flash").treesitter()`.

Co-authored-by: Folke Lemaitre <[email protected]>
myypo pushed a commit to myypo/flash.nvim that referenced this pull request Aug 10, 2024
🤖 I have created a release *beep* *boop*
---


## [2.1.0](folke/flash.nvim@v2.0.0...v2.1.0)
(2024-07-07)


### Features

* **prompt:** add `flash.prompt()` to get the prompt text
([folke#347](folke#347))
([f226e02](folke@f226e02))
* **treesitter:** opt-out selection of the smallest node with
flash.treesitter() by specifying autojump = false
([folke#276](folke#276))
([daa06cd](folke@daa06cd))


### Bug Fixes

* **state:** abort to use Util.exit()
([folke#324](folke#324))
([e7ab1e4](folke@e7ab1e4))
* **treesitter:** process all trees. Closes
[folke#275](folke#275)
([51f3ee1](folke@51f3ee1))

---
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants