Skip to content

feat(watermelon): as 2nd background option main#32

Open
adamchenwei wants to merge 1 commit intoTechForPalestine:mainfrom
adamchenwei:feature/watermelon-background-as-option
Open

feat(watermelon): as 2nd background option main#32
adamchenwei wants to merge 1 commit intoTechForPalestine:mainfrom
adamchenwei:feature/watermelon-background-as-option

Conversation

@adamchenwei
Copy link
Copy Markdown

Screen.Recording.2024-02-07.at.6.24.59.PM.mov

@adamchenwei
Copy link
Copy Markdown
Author

@janajri what you think brotha?

Comment thread src/app/page.tsx
import download from 'downloadjs';
import { toPng } from 'html-to-image';
import Image from 'next/image';
"use client";
Copy link
Copy Markdown
Author

@adamchenwei adamchenwei Feb 8, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

btw I think my VSCode just did a massive formatting refactor on the page... not sure its what you want though 🤔 let me know I can remove pretty much all of them and just leave the code, or if you want just part of the refactor, I can help to clean some of them.

@pbiggar
Copy link
Copy Markdown
Member

pbiggar commented Feb 8, 2024

Surely a 2nd background defeats the purpose

@adamchenwei
Copy link
Copy Markdown
Author

Surely a 2nd background defeats the purpose

how so? 🤔 care to elaborate?

@pbiggar
Copy link
Copy Markdown
Member

pbiggar commented Feb 9, 2024

Surely a 2nd background defeats the purpose

how so? 🤔 care to elaborate?

It seems to me the purpose is to have a common background that is recognizable, and having 2 (or 3 or 4) is less compelling.

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