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

Landing page #44

Merged
merged 30 commits into from
Oct 31, 2024
Merged

Landing page #44

merged 30 commits into from
Oct 31, 2024

Conversation

saraswatpuneet
Copy link
Contributor

@saraswatpuneet saraswatpuneet commented Oct 30, 2024

Description

Please include a summary of the changes and the related issue. Please also include relevant motivation and context.

Screenshot(s)

Including screenshots helps reviewers know what to expect.

Desktop

image

Mobile

image

Type of change

Please delete options that are not relevant.

  • New Feature

How to Test?

Please describe how to test that you ran to verify your changes. Provide instructions so we can reproduce. Please also
list any relevant details for your test configuration

  1. git clone the repository
  2. cd frequency-xyz && npm i
  3. npm run dev then got to http://localhost:3000/docs

Checklist:

  • I have performed a self-review of my code
  • I have commented my code & PR, particularly in hard-to-understand areas
  • I have checked at all the breakpoints to make sure it works on all screen sizes

Sorry, something went wrong.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Copy link

🚀 Preview deployment for this PR is going up! Remember you might need to hard refresh to get the new content.

Preview URL: https://www.frequency.xyz/_pr/44/
Commit: a0da65c

Copy link

🚀 Preview deployment for this PR is going up! Remember you might need to hard refresh to get the new content.

Preview URL: https://www.frequency.xyz/_pr/44/
Commit: 8bcf717

Copy link

🚀 Preview deployment for this PR is going up! Remember you might need to hard refresh to get the new content.

Preview URL: https://www.frequency.xyz/_pr/44/
Commit: 609a904

Copy link

🚀 Preview deployment for this PR is going up! Remember you might need to hard refresh to get the new content.

Preview URL: https://www.frequency.xyz/_pr/44/
Commit: fdbfb2a

Copy link

🚀 Preview deployment for this PR is going up! Remember you might need to hard refresh to get the new content.

Preview URL: https://www.frequency.xyz/_pr/44/
Commit: d4eafd4

@saraswatpuneet saraswatpuneet marked this pull request as ready for review October 30, 2024 20:39
Copy link

🚀 Preview deployment for this PR is going up! Remember you might need to hard refresh to get the new content.

Preview URL: https://www.frequency.xyz/_pr/44/
Commit: fdceef0

Copy link

🚀 Preview deployment for this PR is going up! Remember you might need to hard refresh to get the new content.

Preview URL: https://www.frequency.xyz/_pr/44/
Commit: fd68822

@shannonwells
Copy link
Contributor

Not sure if you meant for this to be reviewed yet, but it does have a lot of issues below 1300px and below 900px width

Copy link

🚀 Preview deployment for this PR is going up! Remember you might need to hard refresh to get the new content.

Preview URL: https://www.frequency.xyz/_pr/44/
Commit: 7a7a9cd

Copy link

🚀 Preview deployment for this PR is going up! Remember you might need to hard refresh to get the new content.

Preview URL: https://www.frequency.xyz/_pr/44/
Commit: af9d971

Copy link

🚀 Preview deployment for this PR is going up! Remember you might need to hard refresh to get the new content.

Preview URL: https://www.frequency.xyz/_pr/44/
Commit: 065c0eb

Copy link

🚀 Preview deployment for this PR is going up! Remember you might need to hard refresh to get the new content.

Preview URL: https://www.frequency.xyz/_pr/44/
Commit: a9d47d6

Copy link
Contributor

@JoeCap08055 JoeCap08055 left a comment

Choose a reason for hiding this comment

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

Couple of typos in the content, but otherwise looks good!

saraswatpuneet and others added 2 commits October 31, 2024 14:18
Co-authored-by: Joe Caputo <joseph.caputo@projectliberty.io>
Co-authored-by: Joe Caputo <joseph.caputo@projectliberty.io>
Copy link

🚀 Preview deployment for this PR is going up! Remember you might need to hard refresh to get the new content.

Preview URL: https://www.frequency.xyz/_pr/44/
Commit: c092222

Copy link

🚀 Preview deployment for this PR is going up! Remember you might need to hard refresh to get the new content.

Preview URL: https://www.frequency.xyz/_pr/44/
Commit: 5a5b57a

Comment on lines 23 to 30
<div class="min-h-[calc(100vh-100px)]">
<div class="relative lg:max-w-[100vw] xl:max-w-page">
<div class="absolute top-0 z-50 w-full"><Header {logoHex} {navColor} bind:section /></div>
<div class={`relative w-[100vw] min-w-full sm:pt-[80px] lg:pt-[140px] vertical-lg:w-[100%]`}>
<DocsBody />
</div>
</div>
</div>
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think you can get away with leaving src/routes/docs/+layout.svelte alone and just doing this:

Suggested change
<div class="min-h-[calc(100vh-100px)]">
<div class="relative lg:max-w-[100vw] xl:max-w-page">
<div class="absolute top-0 z-50 w-full"><Header {logoHex} {navColor} bind:section /></div>
<div class={`relative w-[100vw] min-w-full sm:pt-[80px] lg:pt-[140px] vertical-lg:w-[100%]`}>
<DocsBody />
</div>
</div>
</div>
<div>
<DocsBody />
</div>

Then you can also remove all the other section, color, and such.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I cannot get away with all, it then does not align with landing page, plus header is needed. I remove an outer class which had no effect

Copy link
Collaborator

Choose a reason for hiding this comment

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

You can reset src/routes/docs/+layout.svelte and have the header in there right?

Copy link

🚀 Preview deployment for this PR is going up! Remember you might need to hard refresh to get the new content.

Preview URL: https://www.frequency.xyz/_pr/44/
Commit: cb1d975

Copy link

🚀 Preview deployment for this PR is going up! Remember you might need to hard refresh to get the new content.

Preview URL: https://www.frequency.xyz/_pr/44/
Commit: e78c020

Copy link

🚀 Preview deployment for this PR is going up! Remember you might need to hard refresh to get the new content.

Preview URL: https://www.frequency.xyz/_pr/44/
Commit: 16cf871

Copy link

🚀 Preview deployment for this PR is going up! Remember you might need to hard refresh to get the new content.

Preview URL: https://www.frequency.xyz/_pr/44/
Commit: fad1177

Copy link

🚀 Preview deployment for this PR is going up! Remember you might need to hard refresh to get the new content.

Preview URL: https://www.frequency.xyz/_pr/44/
Commit: a524569

Copy link

🚀 Preview deployment for this PR is going up! Remember you might need to hard refresh to get the new content.

Preview URL: https://www.frequency.xyz/_pr/44/
Commit: fb5f619

Copy link

🚀 Preview deployment for this PR is going up! Remember you might need to hard refresh to get the new content.

Preview URL: https://www.frequency.xyz/_pr/44/
Commit: ac49948

@saraswatpuneet saraswatpuneet merged commit 283af5c into developer-portal Oct 31, 2024
5 checks passed
@saraswatpuneet saraswatpuneet deleted the landing_page branch October 31, 2024 22:07
github-actions bot pushed a commit that referenced this pull request Oct 31, 2024
wilwade pushed a commit that referenced this pull request Nov 19, 2024
# Description

Please include a summary of the changes and the related issue. Please
also include relevant motivation and context.

- Closes #31 

## Screenshot(s)

Including screenshots helps reviewers know what to expect.

### Desktop


![image](https://github.com/user-attachments/assets/30fa5aac-8dcb-4867-bfd0-e87abfabe1b9)


### Mobile

![image](https://github.com/user-attachments/assets/1b200ef1-c584-4a0b-8939-41ef9af294b5)

## Type of change

Please delete options that are not relevant.
- [x] New Feature

# How to Test?

Please describe how to test that you ran to verify your changes. Provide
instructions so we can reproduce. Please also
list any relevant details for your test configuration

1. git clone the repository
2. `cd frequency-xyz && npm i`
3. `npm run dev` then got to `http://localhost:3000/docs`

# Checklist:

- [x] I have performed a self-review of my code
- [ ] I have commented my code & PR, particularly in hard-to-understand
areas
- [ ] I have checked at all the breakpoints to make sure it works on all
screen sizes

---------

Co-authored-by: Joe Caputo <joseph.caputo@projectliberty.io>
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