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

feat(vikunja): init #723

Merged
merged 8 commits into from
Jul 29, 2024
Merged

feat(vikunja): init #723

merged 8 commits into from
Jul 29, 2024

Conversation

Guaxinim5573
Copy link
Member

🎉 Theme for Vikunja 🎉

Vikunja is the open-source, self-hostable to-do app.

latte
frappe
macchiato
mocha

💬 Additional Comments 💬

Note

To add your instance, go to the userstyle's Settings > Custom included sites and add your instance to the list, for example, *://vikunja.example.org/*.

🗒 Checklist 🗒

  • I have read and followed Catppuccin's submission guidelines.
  • I have made a new directory underneath /styles/<name-of-website> containing the contents of the /template directory.
    • I have ensured that the new directory is in lower-kebab-case.
    • I have followed the template and kept the preprocessor as LESS.
  • I have made sure to update the
    userstyles.yml
    file with information about the new userstyle.
  • I have included the following files:
    • catppuccin.user.css - all the CSS for the userstyle, based on the
      template.
    • preview.webp - composite image of all four individual flavor screenshots stitched together,
      generated via Catwalk.

@Guaxinim5573 Guaxinim5573 changed the title feat(init): Vikunja feat(vikunja): init Mar 28, 2024
@isabelroses isabelroses self-requested a review March 28, 2024 07:16
scripts/userstyles.yml Outdated Show resolved Hide resolved
styles/vikunja/catppuccin.user.css Outdated Show resolved Hide resolved
styles/vikunja/catppuccin.user.css Show resolved Hide resolved
styles/vikunja/catppuccin.user.css Show resolved Hide resolved
@isabelroses isabelroses marked this pull request as draft June 8, 2024 14:56
@sgoudham
Copy link
Contributor

Hey @Guaxinim5573 👋

Just checking in to see if you're still interested in contributing the Vikunja userstyle? I couldn't help but notice that changes were requested a while ago and it's gone a bit stale since.

No worries if you haven't had the time! Also if you're no longer interested then that's perfectly okay too. I'd appreciate a reply, thank you!

@Guaxinim5573
Copy link
Member Author

Hey @Guaxinim5573 👋

Just checking in to see if you're still interested in contributing the Vikunja userstyle? I couldn't help but notice that changes were requested a while ago and it's gone a bit stale since.

No worries if you haven't had the time! Also if you're no longer interested then that's perfectly okay too. I'd appreciate a reply, thank you!

After submitting the PR I got a lot of work but then I forgot about it since. I'm really sorry for that. I will retest the userstyle on newer version and include the requested changes in a few days.

@Guaxinim5573 Guaxinim5573 marked this pull request as ready for review July 24, 2024 06:09
@Guaxinim5573 Guaxinim5573 requested a review from a team as a code owner July 24, 2024 06:09
Copy link
Member

@isabelroses isabelroses left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything appears to be themed on my instance. However, i have a issue with the contrasting on most cards and layered content, see the image below, it would be nice to either make the background mantle and make the cards base, which i'm preferential to. Or make the background base and make the cards surface0. Or if we want to follow the originals design pattern base for the background and mantle for the cards.
image

styles/vikunja/catppuccin.user.css Outdated Show resolved Hide resolved
@isabelroses isabelroses merged commit 1c60cdd into catppuccin:main Jul 29, 2024
5 checks passed
@Guaxinim5573 Guaxinim5573 deleted the feat/vikunja branch July 29, 2024 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants