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

refactor(palette): Nightfox bg/sel conform to standard #158

Merged
merged 1 commit into from
May 18, 2022

Conversation

EdenEast
Copy link
Owner

@EdenEast EdenEast commented May 18, 2022

This change brings nightfox inline with other palettes in regards to bg
and sel gradients. These colors where created from this code section:

local C = require("nightfox.lib.color")
local pal = require("nightfox.palette").load("nightfox")

local bg = C(pal.bg1)
local fg = C(pal.fg1)
local blue = C(pal.blue.base)

require("nightfox").setup({
  palettes = {
    nightfox = {
      bg2 = bg:brighten(6):to_css(),
      bg3 = bg:brighten(12):to_css(),
      bg4 = bg:brighten(24):to_css(),
      sel0 = bg:blend(C(pal.blue.base), 0.2):to_css(),
      sel1 = bg:blend(C(pal.blue.base), 0.4):to_css(),
      comment = bg:blend(fg, 0.5):blend(blue, 0.2):to_css(),
    },
  },
})

Old / New

old

new

Resolves: #155
Supersedes: #157

This change brings nightfox inline with other palettes in regards to bg
and sel gradients. These colors where created from this code section:

```lua
local C = require("nightfox.lib.color")
local pal = require("nightfox.palette").load("nightfox")

local bg = C(pal.bg1)
local fg = C(pal.fg1)
local blue = C(pal.blue.base)

require("nightfox").setup({
  palettes = {
    nightfox = {
      bg2 = bg:brighten(6):to_css(),
      bg3 = bg:brighten(12):to_css(),
      bg4 = bg:brighten(24):to_css(),
      sel0 = bg:blend(C(pal.blue.base), 0.2):to_css(),
      sel1 = bg:blend(C(pal.blue.base), 0.4):to_css(),
      comment = bg:blend(fg, 0.5):blend(blue, 0.2):to_css(),
    },
  },
})
```
@EdenEast EdenEast added the palette Change related color values of palette label May 18, 2022
@EdenEast EdenEast self-assigned this May 18, 2022
@EdenEast EdenEast merged commit 320fe16 into main May 18, 2022
@EdenEast EdenEast deleted the fix/nightfox-palette branch May 18, 2022 01:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
palette Change related color values of palette
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Comments are hard to read when cursorline is enabled
1 participant