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(pluralkit): init #1559

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

waterlilly-lilly
Copy link

🎉 Theme for PluralKit Dashboard 🎉

PluralKit Dashboard is a Web-based dashboard for the PluralKit Discord bot.
preview.webp
Screenshot 2025-01-04 at 16-15-11 PluralKit profile
Screenshot 2025-01-04 at 16-14-58 PluralKit dash

💬 Additional Comments 💬

  • I couldn't include any screenshots showing the member or group pages due to licensing concerns with the profile pictures I use. Since these are the main part that a user will interact with, it might be worth finding a solution like creating an example system where this isn't a concern.
  • I had to use a bit of a kludge to color a few UI element SVGs with hardcoded colors (involving the e() function from lesscss)
  • Similar story with the status page, where I had to use an attribute selector on the style attribute for the shard class in order to style the colors.
  • I'm not totally sure what the standard practice for borders is, so I used color offsets from the element background color. This could be pretty easily changed, though.
  • When I was editing it in the builtin Stylus editor, it showed an error on line 25 ("unknown word"). It didn't seem to affect anything, so I'm not sure if it's a problem or not.
  • I could probably theme the main pluralkit.me page with relative ease, though the average PluralKit user interacts with the dashboard much more.
  • I don't think this is too niche, considering that the bot is in nearly a million servers, but I'm not sure what the criteria for nicheness is.

🗒 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.less - 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.

@uncenter
Copy link
Member

uncenter commented Jan 4, 2025

No need to make a new PR in the future please, harder to keep track of previous comments now and we can instead just rename the title and update the body.

scripts/userstyles.yml Outdated Show resolved Hide resolved
styles/pluralkit/catppuccin.user.less Outdated Show resolved Hide resolved
styles/pluralkit/catppuccin.user.less Outdated Show resolved Hide resolved
styles/pluralkit/catppuccin.user.less Outdated Show resolved Hide resolved
styles/pluralkit/catppuccin.user.less Outdated Show resolved Hide resolved
styles/pluralkit/catppuccin.user.less Outdated Show resolved Hide resolved
styles/pluralkit/catppuccin.user.less Outdated Show resolved Hide resolved
}
&-bar {
&:hover {
color: lighten(@subtext0, 5%);
Copy link
Member

Choose a reason for hiding this comment

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

subtext0 should not need to be "lightened", use a different shade like overlay2 or subtext1.

Comment on lines +391 to +505
@blue: #1e66f5;
@lavender: #7287fd;
@text: #4c4f69;
@subtext1: #5c5f77;
@subtext0: #6c6f85;
@overlay2: #7c7f93;
@overlay1: #8c8fa1;
@overlay0: #9ca0b0;
@surface2: #acb0be;
@surface1: #bcc0cc;
@surface0: #ccd0da;
@base: #eff1f5;
@mantle: #e6e9ef;
@crust: #dce0e8;
};
@frappe: {
@rosewater: #f2d5cf;
@flamingo: #eebebe;
@pink: #f4b8e4;
@mauve: #ca9ee6;
@red: #e78284;
@maroon: #ea999c;
@peach: #ef9f76;
@yellow: #e5c890;
@green: #a6d189;
@teal: #81c8be;
@sky: #99d1db;
@sapphire: #85c1dc;
@blue: #8caaee;
@lavender: #babbf1;
@text: #c6d0f5;
@subtext1: #b5bfe2;
@subtext0: #a5adce;
@overlay2: #949cbb;
@overlay1: #838ba7;
@overlay0: #737994;
@surface2: #626880;
@surface1: #51576d;
@surface0: #414559;
@base: #303446;
@mantle: #292c3c;
@crust: #232634;
};
@macchiato: {
@rosewater: #f4dbd6;
@flamingo: #f0c6c6;
@pink: #f5bde6;
@mauve: #c6a0f6;
@red: #ed8796;
@maroon: #ee99a0;
@peach: #f5a97f;
@yellow: #eed49f;
@green: #a6da95;
@teal: #8bd5ca;
@sky: #91d7e3;
@sapphire: #7dc4e4;
@blue: #8aadf4;
@lavender: #b7bdf8;
@text: #cad3f5;
@subtext1: #b8c0e0;
@subtext0: #a5adcb;
@overlay2: #939ab7;
@overlay1: #8087a2;
@overlay0: #6e738d;
@surface2: #5b6078;
@surface1: #494d64;
@surface0: #363a4f;
@base: #24273a;
@mantle: #1e2030;
@crust: #181926;
};
@mocha: {
@rosewater: #f5e0dc;
@flamingo: #f2cdcd;
@pink: #f5c2e7;
@mauve: #cba6f7;
@red: #f38ba8;
@maroon: #eba0ac;
@peach: #fab387;
@yellow: #f9e2af;
@green: #a6e3a1;
@teal: #94e2d5;
@sky: #89dceb;
@sapphire: #74c7ec;
@blue: #89b4fa;
@lavender: #b4befe;
@text: #cdd6f4;
@subtext1: #bac2de;
@subtext0: #a6adc8;
@overlay2: #9399b2;
@overlay1: #7f849c;
@overlay0: #6c7086;
@surface2: #585b70;
@surface1: #45475a;
@surface0: #313244;
@base: #1e1e2e;
@mantle: #181825;
@crust: #11111b;
};
};
Copy link
Member

Choose a reason for hiding this comment

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

Your formatter changed the formatting here as well. Please update it with the contents from the template, and again try to limit your editor to use deno fmt only :)

@uncenter uncenter added the waiting on author Note for staff that a re-review is not yet required label Jan 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
waiting on author Note for staff that a re-review is not yet required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants