userstyles icon indicating copy to clipboard operation
userstyles copied to clipboard
verified publisher icon catppuccin

feat(google-contacts): init

Open WalkQuackBack opened this issue 9 months ago • 15 comments

🎉 Theme for Website 🎉

Google Contacts is a contact management application for G Suite, also known as Google Workspace.

contacts google com_u_1_new_hl=en(1080p) contacts google com_u_1__hl=en(1080p) (1) contacts google com_u_1__hl=en(1080p)

💬 Additional Comments 💬

I struggle with differentiating certain colours, so I may have left certain colour combos that are not very legible together. When reviewing please point them out to me and I will try my best to fix them.

🗒 Checklist 🗒

  • [x] I have read and followed Catppuccin's submission guidelines.
  • [x] I have made a new directory underneath /styles/<name-of-website> containing the contents of the /template directory.
    • [x] I have ensured that the new directory is in lower-kebab-case.
    • [x] I have followed the template and kept the preprocessor as LESS.
  • [x] I have made sure to update the userstyles.yml file with information about the new userstyle.
  • [x] I have included the following files:
    • [x] catppuccin.user.less - all the CSS for the userstyle, based on the template.
    • [x] preview.webp - composite image of all four individual flavor screenshots (taken with the default accent color of mauve) stitched together, generated via Catwalk.

WalkQuackBack avatar Mar 11 '25 03:03 WalkQuackBack

I'll merge in the requested changes regarding the RGB variables and the hardcoded colours in a bit: then we can move onto resolving the accent container colours.

WalkQuackBack avatar Mar 11 '25 21:03 WalkQuackBack

I'll try to test this on various Google Workspace plans later today to see if there's any CSS differences that need to be corrected.

WalkQuackBack avatar Mar 12 '25 18:03 WalkQuackBack

There's some things that aren't themed on higher Google Workspace plans like delegates and the name plate: I'll fix those up then you can review. Could you also look into the Latte style rules not working: I still haven't figured it out and left a comment in the file for you to look at.

WalkQuackBack avatar Mar 12 '25 22:03 WalkQuackBack

Oops: I meant to merge my latest commits first, please wait a few minutes

WalkQuackBack avatar Mar 13 '25 22:03 WalkQuackBack

That should be good.

WalkQuackBack avatar Mar 13 '25 22:03 WalkQuackBack

@uncenter Ready for review!

WalkQuackBack avatar Mar 13 '25 23:03 WalkQuackBack

I noticed that GitHub says that there's requested changes that need to be resolved: however, I'm not actually sure what needs to be changed. Could that be pointed out?

WalkQuackBack avatar Mar 14 '25 23:03 WalkQuackBack

There are no changes requested at the moment, I have not re-reviewed. Until then though please see to the linter errors.

uncenter avatar Mar 14 '25 23:03 uncenter

Never mind, the linter is throwing a fit that I'll see to later. Ignore for now.

uncenter avatar Mar 14 '25 23:03 uncenter

Um

NeonGamerBot-QK avatar Mar 19 '25 04:03 NeonGamerBot-QK

-_- why does GitHub Mobile not have confirmations before it changes stuff like that

WalkQuackBack avatar Mar 19 '25 05:03 WalkQuackBack

Noticed the checkbox for preview.webp was unchecked by a maintainer: is this something requiring my attention?

WalkQuackBack avatar Mar 22 '25 05:03 WalkQuackBack

Nope, that appears to have been a misclick.

uncenter avatar Mar 22 '25 15:03 uncenter

It was mentioned how the generated class names with no comments can be hard to maintain so I added descriptions of what most classes target.

WalkQuackBack avatar Mar 23 '25 22:03 WalkQuackBack

Considering how this app and Google Cloud use the --gm variables quite heavily in legacy areas, might want to consider making a library for them at some point.

WalkQuackBack avatar Sep 25 '25 13:09 WalkQuackBack