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(japanesewithanime.com): init #851

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

Conversation

BPplays
Copy link
Contributor

@BPplays BPplays commented Apr 29, 2024

🎉 Theme for japanesewithanime.com 🎉

image

💬 Additional Comments 💬

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

scripts/userstyles.yml Outdated Show resolved Hide resolved
border: 4px solid @surface0 !important;
border-color: @surface0 !important;
border-width: 4px !important;
border-style: solid !important;
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 setting anything other than border-color?

Copy link
Contributor Author

@BPplays BPplays Apr 29, 2024

Choose a reason for hiding this comment

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

@uncenter i've removed some of the size changes i think are unnecessary but here are some reasons for the remaining ones:

  • for input.gsc-search-button and input.gsc-input i feel like the original border style doesn't work for the theme and it looks odd without the slight rounding to replicate the rounding of the border and the size change to make the search button the same size as the search bar

  • for #archive-year-select and #archive-month-select i feel like a border looks bad and slight rounding replicates the border look in the original theme

  • for .megaNav__Particles > a, .megaNav__Symbols > a i also feel like a border looks bad here and it looks weird without slight rounding

  • for .post-donation-cta same as above

@isabelroses
Copy link
Member

Having taken a look I agree that the website does look wierd when you have the catppuccin colors with those borders, but this is still an opinionated change and i'm unsure wether it should be allowed.

@BPplays
Copy link
Contributor Author

BPplays commented May 25, 2024

@isabelroses i've added a commit with the option to change some of the formatting changes.

  • Prefer solid colors with no borders controls all the stuff exept the search bar i mentioned before and is off by default
  • Original search bar changes the search bar back to the original and is off by default

some things like the search bar button needed a lot of formatting changes to look like the original and i'm not sure why

@isabelroses isabelroses requested a review from uncenter May 25, 2024 08:40
@uncenter uncenter self-assigned this May 25, 2024
@uncenter
Copy link
Member

Can you update this branch with main and resolve the conflicts?

@BPplays BPplays force-pushed the feat/japanesewithanime.com branch from 90d8077 to 0484b87 Compare May 26, 2024 01:21
@BPplays
Copy link
Contributor Author

BPplays commented May 26, 2024

@uncenter it's updated now

Copy link
Member

@uncenter uncenter left a comment

Choose a reason for hiding this comment

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

Also please remove commented out styles and format the document.

@preprocessor less
@var select lightFlavor "Light Flavor" ["latte:Latte*", "frappe:Frappé", "macchiato:Macchiato", "mocha:Mocha"]
@var select darkFlavor "Dark Flavor" ["latte:Latte", "frappe:Frappé", "macchiato:Macchiato", "mocha:Mocha*"]
@var select accentColor "Accent" ["rosewater:Rosewater", "flamingo:Flamingo", "pink:Pink", "mauve:Mauve", "red:Red*", "maroon:Maroon", "peach:Peach", "yellow:Yellow", "green:Green", "teal:Teal", "blue:Blue", "sapphire:Sapphire", "sky:Sky", "lavender:Lavender", "subtext0:Gray"]
Copy link
Member

Choose a reason for hiding this comment

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

See lint annotation.

@var select lightFlavor "Light Flavor" ["latte:Latte*", "frappe:Frappé", "macchiato:Macchiato", "mocha:Mocha"]
@var select darkFlavor "Dark Flavor" ["latte:Latte", "frappe:Frappé", "macchiato:Macchiato", "mocha:Mocha*"]
@var select accentColor "Accent" ["rosewater:Rosewater", "flamingo:Flamingo", "pink:Pink", "mauve:Mauve", "red:Red*", "maroon:Maroon", "peach:Peach", "yellow:Yellow", "green:Green", "teal:Teal", "blue:Blue", "sapphire:Sapphire", "sky:Sky", "lavender:Lavender", "subtext0:Gray"]
@var select linkColor "Link color" ["rosewater:Rosewater", "flamingo:Flamingo", "pink:Pink", "mauve:Mauve", "red:Red", "maroon:Maroon", "peach:Peach", "yellow:Yellow", "green:Green", "teal:Teal", "blue:Blue*", "sapphire:Sapphire", "sky:Sky", "lavender:Lavender", "subtext0:Gray"]
Copy link
Member

Choose a reason for hiding this comment

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

We should use either accent color or blue and lavender as per the style guide.

@var select darkFlavor "Dark Flavor" ["latte:Latte", "frappe:Frappé", "macchiato:Macchiato", "mocha:Mocha*"]
@var select accentColor "Accent" ["rosewater:Rosewater", "flamingo:Flamingo", "pink:Pink", "mauve:Mauve", "red:Red*", "maroon:Maroon", "peach:Peach", "yellow:Yellow", "green:Green", "teal:Teal", "blue:Blue", "sapphire:Sapphire", "sky:Sky", "lavender:Lavender", "subtext0:Gray"]
@var select linkColor "Link color" ["rosewater:Rosewater", "flamingo:Flamingo", "pink:Pink", "mauve:Mauve", "red:Red", "maroon:Maroon", "peach:Peach", "yellow:Yellow", "green:Green", "teal:Teal", "blue:Blue*", "sapphire:Sapphire", "sky:Sky", "lavender:Lavender", "subtext0:Gray"]
@var checkbox solidColors "Prefer solid colors with no borders" 0
Copy link
Member

Choose a reason for hiding this comment

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

Pick one ideally, maybe the one that most matches the original site's appearance.

Comment on lines +137 to +162
& when (@ogBar = 1) {
input.gsc-search-button{
border-color: @overlay1 !important;
border-radius: 2px !important;
border-width: 1px !important;
padding: 1.8px 6.6px !important;
border-style: solid !important;
background-color: @surface1 !important;
}
input.gsc-input{
border-color: @surface1 !important;
}
}
& when (@ogBar = 0) {
input.gsc-search-button{
background-color: @surface1 !important;
border-color: @surface1 !important;
border-width: 1px !important;
border-style: solid !important;
border-radius: 1px !important;
}
input.gsc-input{
border-color: @surface0 !important;
border-radius: 1px !important;
}
}
Copy link
Member

Choose a reason for hiding this comment

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

Just go with one search bar styling I think? Again, whichever better fits the website's original appearance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants