-
-
Notifications
You must be signed in to change notification settings - Fork 98
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
base: main
Are you sure you want to change the base?
Conversation
border: 4px solid @surface0 !important; | ||
border-color: @surface0 !important; | ||
border-width: 4px !important; | ||
border-style: solid !important; |
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.
Why are we setting anything other than border-color
?
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.
@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
andinput.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
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. |
@isabelroses i've added a commit with the option to change some of the formatting changes.
some things like the search bar button needed a lot of formatting changes to look like the original and i'm not sure why |
Can you update this branch with main and resolve the conflicts? |
…and Original search bar
90d8077
to
0484b87
Compare
@uncenter it's updated now |
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.
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"] |
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.
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"] |
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.
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 |
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.
Pick one ideally, maybe the one that most matches the original site's appearance.
& 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; | ||
} | ||
} |
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.
Just go with one search bar styling I think? Again, whichever better fits the website's original appearance.
🎉 Theme for japanesewithanime.com 🎉
💬 Additional Comments 💬
🗒 Checklist 🗒
/styles/<name-of-website>
containing the contents of the/template
directory.userstyles.yml
file with information about the new userstyle.
catppuccin.user.css
- all the CSS for the userstyle, based on thetemplate.
preview.webp
- composite image of all four individual flavor screenshots stitched together,generated via Catwalk.