Refactor #665
Merged
Refactor #665
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…meta_return If you did not use this mapping directly everything is ok. This is just for users that mapped it directly
…g_indent_mode` (#658) * feat: make VirtualIndent react to changes in `vim.b.org_indent_mode` * test: add test to validate VirtualIndent dynamic attach functionality * docs: update docs to reflect dynamic virtual indent attach * refactor: make VirtualIndent `start_watch_org_indent` idempotent This ensures we can call `start_watch_org_indent` as much as we want without starting a bunch of timers in the background. This enforces the use of `start_watch_org_indent` and `stop_watch_org_indent` for managing the timer. * feat: add dict_watcher utility * refactor: use `dict_watcher` to monitor `vim.b.org_indent_mode`
On the current version of nightly (at the time of writing this is NVIM v0.10.0-dev-2361+ga376d979b) the synchronous call doesn't seem to have any value. Using only a scheduled call is seemingly smooth now -- perhaps when I made the original commit I had something wrong with my configuration or perhaps upstream Neovim fixed whatever was causing some choppiness? Another note is that `on_lines` in `nvim_buf_attach` can have `textlock` restrictions and more applied, thus we really should just use `vim.schedule` only anyhow.
* feat: add equivalent of `org-indent-mode-turns-on-hiding-stars` * docs: add documentation for `org_indent_mode_turns_on_hiding_stars` * docs: add reference to `org_indent_mode` for hiding stars Co-authored-by: Kristijan Husak <husakkristijan@gmail.com> * fix: ensure hl group for hiding stars is applied See #659 (review) Co-authored-by: Kristijan Husak <husakkristijan@gmail.com> --------- Co-authored-by: Kristijan Husak <husakkristijan@gmail.com>
This ensures that notifications do not error on startup. Prior to this commit the `OrgFiles` object wasn't loaded in time for notifications to work.
SlayerOfTheBad
pushed a commit
to SlayerOfTheBad/orgmode
that referenced
this pull request
Aug 16, 2024
* refactor!: Rewrite file management to use tree-sitter * feat!: Deprecate org_mappings.handle_return in favor of org_mappings.meta_return If you did not use this mapping directly everything is ok. This is just for users that mapped it directly * feat: Add org_return_uses_meta_return option * fix: Allow modifications of plan date range end date * fix: Properly show plan date ranges in agenda * feat(api): Add get_closest_headline and refile to api * fix: Update minimal init to work with Windows * tests: Add tests for file and fix edge case with set text * fix(cron): Instantiate orgmode before running cron * refactor: Stars highlighter * refactor: Todo faces highlighter * refactor: Markup highlighter * fix: Do not rely on node:has_changes() * feat: make VirtualIndent dynamically attach/detach based on `vim.b.org_indent_mode` (nvim-orgmode#658) * feat: make VirtualIndent react to changes in `vim.b.org_indent_mode` * test: add test to validate VirtualIndent dynamic attach functionality * docs: update docs to reflect dynamic virtual indent attach * refactor: make VirtualIndent `start_watch_org_indent` idempotent This ensures we can call `start_watch_org_indent` as much as we want without starting a bunch of timers in the background. This enforces the use of `start_watch_org_indent` and `stop_watch_org_indent` for managing the timer. * feat: add dict_watcher utility * refactor: use `dict_watcher` to monitor `vim.b.org_indent_mode` * ci: Run tests when creating PR to nightly * chore: Reformat files * refactor: remove sync call to set indent for virtual indents (nvim-orgmode#663) On the current version of nightly (at the time of writing this is NVIM v0.10.0-dev-2361+ga376d979b) the synchronous call doesn't seem to have any value. Using only a scheduled call is seemingly smooth now -- perhaps when I made the original commit I had something wrong with my configuration or perhaps upstream Neovim fixed whatever was causing some choppiness? Another note is that `on_lines` in `nvim_buf_attach` can have `textlock` restrictions and more applied, thus we really should just use `vim.schedule` only anyhow. * feat: add `org-indent-mode-turns-on-hiding-stars` equivalent (nvim-orgmode#659) * feat: add equivalent of `org-indent-mode-turns-on-hiding-stars` * docs: add documentation for `org_indent_mode_turns_on_hiding_stars` * docs: add reference to `org_indent_mode` for hiding stars Co-authored-by: Kristijan Husak <husakkristijan@gmail.com> * fix: ensure hl group for hiding stars is applied See nvim-orgmode#659 (review) Co-authored-by: Kristijan Husak <husakkristijan@gmail.com> --------- Co-authored-by: Kristijan Husak <husakkristijan@gmail.com> * fix: Hide leading stars on foldtext * ci: Run tests only on nightly * fix: run Org init as part of setup (nvim-orgmode#664) This ensures that notifications do not error on startup. Prior to this commit the `OrgFiles` object wasn't loaded in time for notifications to work. * fix: Statusline function for clock * ci: Remove nightly branch from CI triggers --------- Co-authored-by: Price Hiller <price@orion-technologies.io>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This branch contains a lot of changes, most notably: