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(myanimelist): init #1401

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

Conversation

orangci
Copy link
Member

@orangci orangci commented Oct 26, 2024

🎉 Theme for MyAnimeList 🎉

MyAnimeList is an indexing site for anime and manga. Users can use it to look up anime, manga, donghua, light novels, aeni, etc, as well as log and keep track of their progress.

💬 Additional Comments 💬

This userstyle does not theme user animelists/mangalists, as many users theme their anime/mangalists with custom CSS.

🗒 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 (taken with the default accent color of mauve) stitched together, generated via Catwalk.

@orangci orangci requested a review from a team as a code owner October 26, 2024 06:26
@orangci orangci marked this pull request as draft October 26, 2024 06:26

@-moz-document domain('myanimelist.net') {
/* Exclude animelist and mangalist pages */
@-moz-document regexp("^(?!.*myanimelist\\.net\\/(animelist|mangalist)\\/).*$") {
Copy link
Member

Choose a reason for hiding this comment

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

Why are we not using this regex instead of the domain(...) above?

Copy link
Member Author

Choose a reason for hiding this comment

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

oh good point

@orangci
Copy link
Member Author

orangci commented Nov 21, 2024

i am busy; i may not work on this for another month .. should i close this PR until then, or leave it open?

@uncenter
Copy link
Member

i am busy; i may not work on this for another month .. should i close this PR until then, or leave it open?

No worries at all. It's okay to keep it as a draft for now :)

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.

2 participants