hyper

hyper copied to clipboard

As per the subject, I've tried with and without version numbers, with no success.

can you please explain what are you trying to do and also paste the whole error

The error occurs on the latest 4.x.x canary builds, probably because of a breaking change somewhere.

The error occurs on the latest 4.x.x canary builds, probably because of a breaking change somewhere.

will look into it

@chrionix @sharpenedblade i dont think i can do anything about it for now, i doubt that it may break things for people not using the canary build, do you guys have any suggestions?

moreover, documentation for the canary build is nowhere to be found :(

Actually it sometimes happens on the latest stable.

moreover, documentation for the canary build is nowhere to be found :( The only docs are in the release notes for now.

I am using macos, could it be related.

I am using macos, could it be related.

not really sure, i use windows and haven't got any errors 🤔 will need more context on this

closed by mistake, will leave this open so that more people can report this

So I tried to get logs, but when I open the devtools console, I dont see any errors.

After some investigation, I realized that the error happens when catppuccinTheme is not set in your config. The canary build moved to json config, so it could be related.

The error comes from

const theme = palettes[config.catppuccinTheme.toLowerCase() || "macchiato"];

in decorateConfig in index.js

After messing with the new config file, I discovered that setting catppuccinTheme in the config file, then restarting hyper makes this plugin work. I think the README should tell people that they have to set catppuccinTheme, because right now its makes it seem optional.

After messing with the new config file, I discovered that setting catppuccinTheme in the config file, then restarting hyper makes this plugin work. I think the README should tell people that they have to set catppuccinTheme, because right now its makes it seem optional.

alright, thanks a lot for helping out, really appreciate this ❤️

@chrionix can you check and let us know if you have set the catppuccinTheme in your config file?

I think this might solve the issue -

const themeName = config.catppuccinTheme ? config.catppuccinTheme.toLowerCase() : "macchiato";
const theme = palettes[themeName];

previously I was doing -

const theme = palettes[config.catppuccinTheme.toLowerCase() || "macchiato"];

the thing is, in the stable builds even not setting the catppuccinTheme works, not really sure what they are doing differently in canary version

the thing is, in the stable builds even not setting the catppuccinTheme works, not really sure what they are doing differently in canary version

It doesn't work in the latest stable for me. Are you using 3.4.1?

Just wanted to note that I get the same error and that adding catppuccinTheme to the config file does make the error go away.

Also confirming that adding catppuccinTheme to the config fixes this for Hyper 3.4.1.

I would update https://github.com/catppuccin/hyper/blob/8281b8f7f872c57308614de741da987dc119a71d/index.js#L107

to

  const theme =  palettes[config.catppuccinTheme ? config.catppuccinTheme.toLowerCase()] || palettes.macchiato;

Hello, I just added the plugin to Hyper but it is not working... I added decorateconfig manually to index.js but my terminal is still using the default theme...

Hey 👋,

I've merged #15 which overhauls the repository and modifies the installation to only apply the colours and not touch the layout/CSS. Please see the README for the new installation instructions.