Skip to content
This repository has been archived by the owner on Apr 16, 2024. It is now read-only.

[Fix] [doom-dashboard.lua #345] #346

Merged
merged 1 commit into from
Jun 24, 2022
Merged

[Fix] [doom-dashboard.lua #345] #346

merged 1 commit into from
Jun 24, 2022

Conversation

cleong14
Copy link
Contributor

(Bug Fixes) - changes made following dashboard-nvim Vimscript -> Lua rewrite

  • add local db = require("dashboard")
  • rewrite (Vimscript) vim.g.dashboard_custom_section -> (Lua) db.custom_center
  • rewrite (Vimscript) vim.g.dashboard_custom_footer -> (Lua) db.custom_footer
  • rewrite (Vimscript) vim.g.dashboard_custom_header -> (Lua) db.custom_header
  • fix custom_header support for when user sets config.doom.dashboard_custom_header
  • fix/update header colors (highlight groups)

(Bug Fixes) - changes made following [`dashboard-nvim`](https://github.com/glepnir/dashboard-nvim) Vimscript -> Lua rewrite

- add `local db = require("dashboard")`
- rewrite (Vimscript) `vim.g.dashboard_custom_section` -> (Lua) `db.custom_center`
- rewrite (Vimscript) `vim.g.dashboard_custom_footer` -> (Lua) `db.custom_footer`
- rewrite (Vimscript) `vim.g.dashboard_custom_header` -> (Lua) `db.custom_header`
- fix `custom_header` support for when user sets `config.doom.dashboard_custom_header`
- fix/update header colors (highlight groups)
@NTBBloodbath NTBBloodbath added branch: main For stuff regarding to main branch plugins: breaking changes Plugins that had breaking changes and needs to be updated priority: high High priority stuff scope: enhancement New feature or request labels Jun 14, 2022
@NTBBloodbath NTBBloodbath linked an issue Jun 14, 2022 that may be closed by this pull request
@cleong14
Copy link
Contributor Author

Hi @NTBBloodbath ,

I reviewed the failing check and it appears to be related to pre-existing code found here

https://github.com/NTBBloodbath/doom-nvim/blob/9e93d6d0e3c9d46909d1797d8ef49a005a02286a/lua/colors/doom-one/utils/init.lua#L88

and here

https://github.com/NTBBloodbath/doom-nvim/blob/9e93d6d0e3c9d46909d1797d8ef49a005a02286a/lua/colors/utils/init.lua#L88

Let me know how I may assist or how you'd like to proceed?

Thanks!

@NextAlone
Copy link
Contributor

NextAlone commented Jun 21, 2022

I combine your changes in this NextAlone@4cd7905

Ohh, this is for next branch, not main branch

@NextAlone
Copy link
Contributor

the lua dir structure has been modified

@connorgmeehan connorgmeehan merged commit 539a1e4 into doom-neovim:main Jun 24, 2022
@connorgmeehan
Copy link
Collaborator

Just tested it all out, seems to be working well. Thanks for the contribution!

@NextAlone
Copy link
Contributor

I combine your changes in this NextAlone@4cd7905

Ohh, this is for next branch, not main branch

@connorgmeehan maybe you see this for next branch

@connorgmeehan
Copy link
Collaborator

Hey nextalone, it should be merged in #353

@NextAlone
Copy link
Contributor

NextAlone commented Jun 24, 2022

Hey nextalone, it should be merged in #353

Oops i forgot the pull request

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
branch: main For stuff regarding to main branch plugins: breaking changes Plugins that had breaking changes and needs to be updated priority: high High priority stuff scope: enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Dashboard broken after upstream update
4 participants