fix: colours in diff codeblock

[CoralPink]

Hello!!
I always use mdbook-catppuccin! 🐈‍⬛

After updating to v1.0.0, I noticed that the color theme is not applied correctly to the diff.

All we are doing in this PR is reverting the parameters related to diffs back to what they were before v1.0.0, but I think this will fix this issue.

Could you please confirm this?

Thanks!! ☕

[nekowinston]

Hey, thanks for the PR.
This is an issue with the highlightjs port that got fixed recently.
The files you edited are generated, please don't edit them; we just need to bump the @catppuccin/highlightjs dependency.

[CoralPink]

Ah, I see. I'm beginning to understand!

Is it my understanding that this can be solved by simply rewriting pallete/package.json?

I'm not sure if my understanding is correct, as I can't get the build to work in my local environment...

> [email protected] build
> sass -I node_modules --no-charset --no-source-map catppuccin.scss:../src/bin/assets/catppuccin.css catppuccin-admonish.scss:../src/bin/assets/catppuccin-admonish.css

Error: Can't find stylesheet to import.
  ╷
1 │ @use "@catppuccin/palette/scss/catppuccin" as catppuccin;
  │ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  ╵
  catppuccin.scss 1:1  root stylesheet

Error: Can't find stylesheet to import.
  ╷
3 │ @use "@catppuccin/palette/scss/catppuccin" as catppuccin;
  │ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  ╵
  catppuccin-admonish.scss 3:1  root stylesheet

[CoralPink]

Sorry, this was because I did not npm install.

Also, looking closer, maybe the package is supposed to be updated on release.

Sorry for the strange English!
You can also dismiss this PR if it gets in the way!

[nekowinston]

No worries, thanks for adjusting the PR! I think that should be it.

[sgoudham]

Hey there 👋, thanks for the pull request!

The changes with the dependency bump look good, could you just help me understand why 051a7ff was reverted in 1d10d5b (#67)? I specified the files because using . was copying over the node_modules directory into the source directory which wasn't great when it came time to cargo publish.

Reworked.

[CoralPink]

Sorry for repeating myself...

In the revert, I thought I had only undone the operations I had done, but I may have been wrong.

I'm getting kind of confused....

This is a very selfish question, but what should I do in this case...?

[CoralPink]

I have tried many things,

The cappuccin.css seems to match the one generated via
npm run build and the current one (which I manually rewrote).

Now all I have to do is fix the mistake in package.json?

...Please squash the messy commit history 😖

[sgoudham]

No need to apologise and don't worry about the commit history!

I tested out the PR locally and it looks great! I appreciate you spending the time and effort to contribute to this repo ❤️

[CoralPink]

Thanks for the kind words!

I was actually in a hurry for nothing as I was doing it in the time before bed....

It was a good experience 😅
Thanks again 🩷