scottmckendry / cyberdream.nvim

🤖💤 High-contrast, Futuristic & Vibrant Coloursheme for Neovim
MIT License
717 stars 30 forks source link

Feedback for Improvement #4

Closed scottmckendry closed 10 months ago

scottmckendry commented 10 months ago

Hello @ianchesal

I came across a recent commit to your dotfiles repo via GitHub search this morning that caught my interest:

https://github.com/ianchesal/dotfiles/commit/329fa7831a0e479d8b0e102081f12d9c100da6f7

I was wondering if I could get your thoughts and feedback. Is there anything you feel was missing from the colour scheme? Anything you would change or not quite right?

Appreciate your time!

Cheers, Scott

ianchesal commented 10 months ago

Oh hey. This is...uh...very direct!

The biggest issue, for me, was the default colour and styling on the mini-tabline.nvim bars. The pink tabline bar was too much for what I need. Garish? Overly prominent for the active indent level?

And for strings, a more muted option other than lime green would be nice. In string-heavy files it got pretty green.

Styling on Telescope windows also seemed possibly broken? Here's what I see with cyberdream:

image

Versus Tokyo Night:

image

The blocking on the modal in Tokyo Night definitely makes it feel a little more polished, to me.

Hope that helps!

Like I said, I'll keep an eye on it. The Philip K. Dick'ness of it was appealing. Just need a scan to tell if I'm a replicant or not before I can edit files, maybe?

ianchesal commented 10 months ago

Oh. Maybe I should try hide_fillchars = false? Might make the modals a little more tokyo night like?

Edit: nope. No difference.

scottmckendry commented 10 months ago

This is excellent feedback! Thank you very much.

The telescope theming is intentional, as I'm not much of a fan of the bordered look that it ships with. That being said, it looks slightly better with the tweaks I have in my config:

image

I'll add an option to toggle the "borderless" look. I'll also add an option to override specific highlight groups (like "String"). I'm thinking something similar to how Kanagawa does it.

Thanks again for the suggestions! Updates coming soon 🚀

scottmckendry commented 10 months ago

@ianchesal I've merged the changes into main. I've included and example config below that should address some or all of the issues you raised.

require("cyberdream").setup({
    transparent = true,
    italic_comments = true,
    hide_fillchars = true,
    borderless_telescope = false, -- more default-looking telescope

    theme = {
        colors = {
            -- Use a slightly darker green
            green = "#06c258",
        },

        highlights = {
            -- Set indent scope to blue (instead of pink)
            MiniIndentScopeSymbol = { fg = "#5ea1ff" },
        },
    },
})

More info on the new options can be found in the Configuring section of the readme.

ianchesal commented 10 months ago

@scottmckendry I'll check it out today.

I should say: you shouldn't do things based on one person's feedback, especially mine! I'm chuffed you did this, but don't sweat my notes. I'm nobody and this is your theme.

Thanks!

Edit: that said...this is pretty fly! Will run it as my daily driver for a few weeks!

image

image