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

Feedback for Improvement #4

Closed
scottmckendry opened this issue Dec 27, 2023 · 5 comments · Fixed by #6
Closed

Feedback for Improvement #4

scottmckendry opened this issue Dec 27, 2023 · 5 comments · Fixed by #6
Assignees
Labels
enhancement New feature or request

Comments

@scottmckendry
Copy link
Owner

Hello @ianchesal

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

ianchesal/dotfiles@329fa78

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

@scottmckendry scottmckendry self-assigned this Dec 27, 2023
@scottmckendry scottmckendry added the enhancement New feature or request label Dec 27, 2023
@ianchesal
Copy link
Contributor

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
Copy link
Contributor

ianchesal commented Dec 27, 2023

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

Edit: nope. No difference.

@scottmckendry
Copy link
Owner Author

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
Copy link
Owner Author

scottmckendry commented Dec 28, 2023

@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
Copy link
Contributor

ianchesal commented Dec 28, 2023

@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

ianchesal added a commit to ianchesal/dotfiles that referenced this issue Dec 28, 2023
Theme designer scottmckendry contacted me about my commit where I undid
using cyberdream: scottmckendry/cyberdream.nvim#4

Our conversation led to some tweaks and this feels more inline with my
Tokyo Night stuff but with his colour vision. Dig it.

Will run it for a few weeks and see how it goes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants