-
-
Notifications
You must be signed in to change notification settings - Fork 40
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
fix: repo is now template conform #24
Conversation
README.md
Outdated
# - ~/.config/alacritty/catppuccin/catppuccin-frappe.yml | ||
# - ~/.config/alacritty/catppuccin/catppuccin-latte.yml | ||
- ~/.config/alacritty/catppuccin/themes/catppuccin-mocha.yml | ||
# - ~/.config/alacritty/catppuccin/themes/atppuccin-macchiato.yml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
attpuccin
D:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
âttpùccín 🥖🥐
I'll push a fix the next week
README.md
Outdated
@@ -12,9 +12,27 @@ | |||
</p> | |||
|
|||
<p align="center"> | |||
<img src="assets/screenshot.webp"/> | |||
<img src="https://raw.githubusercontent.com/catppuccin/alacritty/main/assets/preview.webp"/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe that hard-linking the preview images is a mistake. The preview on the template repo is only hard-linked because the image itself is from a completely different repo altogether.
Port-readmes usually just softlink the previews. c:
<p align="center">
<img src="assets/preview.webp"/>
</p>
## Previews
<details>
<summary>🌻 Latte</summary>
<img src="assets/latte.webp"/>
</details>
<details>
<summary>🪴 Frappé</summary>
<img src="assets/frappe.webp"/>
</details>
<details>
<summary>🌺 Macchiato</summary>
<img src="assets/macchiato.webp"/>
</details>
<details>
<summary>🌿 Mocha</summary>
<img src="assets/mocha.webp"/>
</details>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've heard both over time, we should declare that somewhere, but changes are on their way non-the less :)
@andreasgrafen added your changes (with a week delay :o) |
Some light work for hacktoberfest ;)