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

feat!: autostart option #104

Merged
merged 10 commits into from
Jul 16, 2022
Merged

feat!: autostart option #104

merged 10 commits into from
Jul 16, 2022

Conversation

bennypowers
Copy link
Contributor

@bennypowers bennypowers commented Apr 24, 2022

Users may set { opts = { autostart = false } } to prevent alpha from drawing on VimEnter

BREAKING CHANGES:
Refactors alpha.lua to use neovim 0.7+ APIs

See #51
Closes #66
Fixes #106

@bennypowers bennypowers changed the title Update alpha.lua feat!: autostart option Apr 24, 2022
Users may set `{ opts = { autostart = false } }` to prevent alpha from drawing on `VimEnter`

BREAKING CHANGES:
Refactors alpha.lua to use neovim 0.7+ APIs
@goolord
Copy link
Owner

goolord commented Jul 12, 2022

it looks like neovim/neovim#14670 was fixed at some point, so I'll look over this once i get some time

Shivix and others added 7 commits July 16, 2022 14:30
The way `end_ln` was incremented before `alpha.highlight` meant that two
calls to `vim.api.nvim_buf_add_highlight` were made for each text line
(i.e. two highlight groups for each line rather than one), causing
highlighting to affect the subsequent line.
Improper handling of `nil` `opts` caused an early conditional
short-circuit that prevented the text line from being padded at all.
@goolord goolord merged commit c79c7a6 into goolord:main Jul 16, 2022
goolord added a commit that referenced this pull request Jul 16, 2022
* feat!: autostart option

Users may set `{ opts = { autostart = false } }` to prevent alpha from drawing on `VimEnter`

BREAKING CHANGES:
Refactors alpha.lua to use neovim 0.7+ APIs

* style: refactors aucmd callbacks to pass state

* fix(debug): incorrect argument

* fix padding.left for text

* remove print

* use autocmd api, don't store old window options

Co-authored-by: Zachary Churchill <[email protected]>
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.

Alpha sets window options, overriding lua config
5 participants