Skip to content

Revert "Feat green day"#2846

Open
atharvadeosthale wants to merge 2 commits intomainfrom
revert-2843-feat-green-day
Open

Revert "Feat green day"#2846
atharvadeosthale wants to merge 2 commits intomainfrom
revert-2843-feat-green-day

Conversation

@atharvadeosthale
Copy link
Copy Markdown
Member

Reverts #2843

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Apr 1, 2026

Greptile Summary

This PR reverts #2843 ("Feat green day"), which added an April Fool's green branding theme to the Appwrite website. The revert cleanly restores the default pink brand identity across all 45 changed files.

Key changes undone by this revert:

  • Theme switching infrastructure — removes BrandTheme type, applyBrandTheme/getPreferredBrandTheme helpers, localStorage persistence, PUBLIC_BRAND_THEME env var support, and MutationObserver-based class-watching in both Main.svelte and Docs.svelte
  • CSS variables for theming — removes --web-color-button-primary, --web-color-brand-rgb, --web-color-image-accent-filter, --web-color-image-background-filter, and all body.brand-green / body.brand-pink overrides; restores hardcoded pink rgba/hsl values in button, badge, card, hero-banner, and side-nav SCSS
  • Image tinting CSS — removes the large block of :global(body.brand-green …) CSS filters in +layout.svelte that recolored icons and background images
  • Green logos — deletes appwrite-green.svg, appwrite-light-green.svg, and logo-green.svg; restores static logo paths in nav headers
  • Brandable wrappers — removes homepage-brandable, products-brandable, docs-brandable, and pricing-brandable container divs used to scope image filters
  • mongo icon — removes the MongoDB SVG icon that was added to the sprite sheet for the green day feature

A Grep search confirmed no residual references to brand-green, brand-pink, BrandTheme, brandTheme, the deleted logo files, or any of the removed CSS variables remain anywhere in src/.

Confidence Score: 5/5

  • Clean, complete revert with no residual green-theme references — safe to merge
  • All 45 files are straightforward reversions of the green day feature. A full-codebase search confirmed zero remaining references to brand-green classes, the removed CSS variables, deleted SVG logos, or brand-switching logic. No new logic was introduced — only removals and restoration of prior hardcoded values.
  • No files require special attention

Important Files Changed

Filename Overview
src/routes/+layout.svelte Removes BrandTheme type, applyBrandTheme/getPreferredBrandTheme helpers, MutationObserver for brand class changes, and all CSS filter rules for green image tinting — cleanly undoes the green day theming infrastructure
src/app.html Removes brand-green default class from body and the localStorage brand-theme initialization script
src/app.css Removes brand-green and brand-pink CSS overrides, restores default pink color hue variables
src/scss/1-css-variables/_colors.scss Removes brand helper CSS variables (--web-color-brand-rgb, --web-color-button-primary, image filter vars) and the body.brand-pink / body.brand-green overrides
src/scss/6-elements/_button.scss Replaces --web-color-button-primary CSS variable references with hardcoded rgba(253, 54, 110, …) pink values, restoring pre-green-day state
src/scss/abstract/mixins/_checked-badge.scss Restores the pre-green-day checked badge (24px, circle+checkmark embedded in SVG) replacing the CSS-driven circular border approach introduced for theming
src/lib/layouts/Main.svelte Removes brandTheme state, dynamic logo sources, brand MutationObserver, isProductsPage variable, and products-brandable class from main element
src/lib/layouts/Docs.svelte Removes brandTheme reactive state, derived logo sources, and brand MutationObserver; restores static appwrite.svg / appwrite-light.svg logo paths
src/lib/components/ui/icon/sprite/sprite.svelte Removes the mongo SVG icon symbol that was added as part of the green day feature
static/images/logos/appwrite-green.svg Green variant of the Appwrite logo deleted as part of the revert

Reviews (2): Last reviewed commit: "Merge branch 'main' into revert-2843-fea..." | Re-trigger Greptile

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants