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(landing): Implement Landing Page for Copod #7

Closed

Conversation

Iamanshuaditya
Copy link

This pull request addresses the issue #1, which involves creating the landing page for Copod, a managed podcast recording studio located in New Delhi.
Refs: #1

@code-withAshish
Copy link
Collaborator

Please attach a screenshot showcasing your entire design, that way it's easier for us to judge it.
Thank you

@Iamanshuaditya
Copy link
Author

Hey @code-withAshish here the deployed link please have a look -https://copod-web.vercel.app/

@rohitt-gupta
Copy link
Collaborator

Hi @Iamanshuaditya ,
PR looks good.

As mentioned in the issue, it is 10$ bounty, will not be increased!

there are few issues in the your deployed version , please fix them!

  1. Make the landing page like modern designs which contains centered CTA buttons and H! just like @iAbhinav mentioned in below thread Landing Page #1 (comment)

  2. Set a limit to max width in tailwind it may be max-w-7xl or 8xl , and mx-auto to make it centered. Do not cover full width of the window/parent container.

  3. Footer is broken and moves along with the view
    image

  4. Use less fancy animations. Too much animations are being used!

  5. Could use the header / nav as fixed!

These some of the suggestions which may make your solution more modern.

Try to take some inspiration from ui.aceternity.com

@iAbhinav
Copy link
Contributor

iAbhinav commented May 8, 2024

Hey guys, copod is a "Venue" for podcasts, and not a podcast itself. We do not have episodes or voices

@code-withAshish
Copy link
Collaborator

as landing page is actively being developed by @iAbhinav and initial version has been merged in the main branch this PR is stale now , hence I am closing it.

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.

4 participants