Skip to content

feat(homepage): Reverting certain changes - #916

Merged
Gandalf-the-Blue merged 1 commit into
mainfrom
homepage_revert
May 25, 2024
Merged

feat(homepage): Reverting certain changes#916
Gandalf-the-Blue merged 1 commit into
mainfrom
homepage_revert

Conversation

@Gandalf-the-Blue

Copy link
Copy Markdown
Contributor

馃敡 What does this fix? 馃敡

Fixes certain issues from PR #915.
Base background colour revert to slightly lighter colour(base), in accordance with catppuccin userstyle guide.
Base box colour revert to slightly darker colour (mantle), in accordance with catppuccin userstyle guide.
Reverted hover colours to accent colour, else accent colour is completely absent from the styling.

Invite @BPplays (original PR author) for any comments.

馃棐 Checklist 馃棐

  • I have read and followed Catppuccin's contributing guidelines.
  • I have updated the version appropriately in the ==UserStyle== header of the catppuccin.user.css file.

@github-actions github-actions Bot added the homepage homepage label May 25, 2024
@Gandalf-the-Blue Gandalf-the-Blue changed the title Reverting certain changes feat(homepage): Reverting certain changes May 25, 2024
@Gandalf-the-Blue Gandalf-the-Blue self-assigned this May 25, 2024
@Gandalf-the-Blue
Gandalf-the-Blue merged commit 8519ff1 into main May 25, 2024
@Gandalf-the-Blue
Gandalf-the-Blue deleted the homepage_revert branch May 25, 2024 19:30
@BPplays

BPplays commented Oct 6, 2024

Copy link
Copy Markdown
Contributor

@Gandalf-the-Blue sorry for the late response,
i changed the background color to mantle and the box color to base because i didn't like the boxes being darker,
and i think surface0 for the boxes, and surface1 for the things on top of the boxes like number of movies looks bad. it also seems to have bad contrast on the numbers with no other style changes.

i changed the accent color highlight because it also affects stuff like number of tv shows, and stats like that but it has a weird dark color on those elements

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

Labels

homepage homepage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants