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(rentry.co): init #1227

Merged
merged 41 commits into from
Oct 12, 2024
Merged

feat(rentry.co): init #1227

merged 41 commits into from
Oct 12, 2024

Conversation

thismoon
Copy link
Contributor

🎉 Theme for Website 🎉

Rentry.co is a markdown paste service with preview, custom urls and editing.¹

💬 Additional Comments 💬

i opened a new pr because renaming the branch closed the old one (#1198). i should've done some research before doing so.

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

@thismoon thismoon requested a review from a team as a code owner August 22, 2024 16:53
@thismoon
Copy link
Contributor Author

thismoon commented Aug 22, 2024

what's wrong with Typo Checker / Spellcheck checking for files i didn't touch?
i guess pulling from the main branch will fix it? would this break the pr?
i think i'll ignore it for now­­

apparently it's a common issue?

@thismoon
Copy link
Contributor Author

@uncenter regarding your review. thanks for suggesting shiki.style. i have added comments to explain the syntax highlighting styles. but there are innaccuracies that i couldn't fix because elements that should have different colors are using the same class. for example

name = "test"
name.len()

image
image
name (a variable) and len() (a function) are using the same class (.n). i'm not sure if i can do something about this

@uncenter
Copy link
Member

Yeah there isn't anything you can do about that, just means Rentry.co is using some subpar highlighter. Thanks for doing those fixes, I'll review later.

@uncenter
Copy link
Member

Omar see #1198 (comment) for the custom text colors. Mark is already themed.

styles/rentry.co/catppuccin.user.css Outdated Show resolved Hide resolved
isabelroses
isabelroses previously approved these changes Sep 3, 2024
styles/rentry.co/catppuccin.user.css Outdated Show resolved Hide resolved
styles/rentry.co/catppuccin.user.css Outdated Show resolved Hide resolved
styles/rentry.co/catppuccin.user.css Outdated Show resolved Hide resolved
styles/rentry.co/catppuccin.user.css Outdated Show resolved Hide resolved
Co-authored-by: uncenter <[email protected]>
isabelroses
isabelroses previously approved these changes Sep 25, 2024
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.

.text-warning-darker is unthemed: put in def as the URL for the doc and it gives a "Entry with this url already exists." warning.

.modal-content>.modal-header is unthemed - try deleting a thingy, the header of the modal that pops up is unthemed.

styles/rentry.co/preview.webp Outdated Show resolved Hide resolved
styles/rentry.co/catppuccin.user.css Outdated Show resolved Hide resolved
uncenter
uncenter previously approved these changes Oct 11, 2024
isabelroses
isabelroses previously approved these changes Oct 11, 2024
@uncenter
Copy link
Member

Oop, let me check #1227 (comment) first.

@uncenter
Copy link
Member

Could you remove that line height modification now that Pygments has updated to remove that rule?

@thismoon thismoon dismissed stale reviews from isabelroses and uncenter via bc2df04 October 12, 2024 12:55
@uncenter uncenter merged commit 7d9f5a7 into catppuccin:main Oct 12, 2024
4 checks passed
@thismoon thismoon deleted the feat/rentry.co branch October 12, 2024 17:06
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.

4 participants