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(watermelon): as 2nd background option main #32

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

adamchenwei
Copy link

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

@adamchenwei
Copy link
Author

@janajri what you think brotha?

import download from 'downloadjs';
import { toPng } from 'html-to-image';
import Image from 'next/image';
"use client";
Copy link
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
Collaborator

pbiggar commented Feb 8, 2024

Surely a 2nd background defeats the purpose

@adamchenwei
Copy link
Author

Surely a 2nd background defeats the purpose

how so? 🤔 care to elaborate?

@pbiggar
Copy link
Collaborator

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