Skip to content

feat: video-landing-page #511

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

Merged
merged 32 commits into from
May 1, 2025
Merged

feat: video-landing-page #511

merged 32 commits into from
May 1, 2025

Conversation

vishalvivekm
Copy link
Contributor

Notes for Reviewers

This PR fixes #

Signed commits

  • Yes, I signed my commits.

Copy link

netlify bot commented Apr 28, 2025

Deploy Preview for bejewelled-pegasus-b0ce81 ready!

Name Link
🔨 Latest commit a450dcb
🔍 Latest deploy log https://app.netlify.com/sites/bejewelled-pegasus-b0ce81/deploys/6813f1afb3e6250008cde94c
😎 Deploy Preview https://deploy-preview-511--bejewelled-pegasus-b0ce81.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@vishalvivekm
Copy link
Contributor Author

vishalvivekm commented Apr 28, 2025

dynamic category, subcategory and video-grid based on folder structure

vishalvivekm@Laptop-LX:~/opensource/layer5/docs/content/en/video-content$ tree
.
├── features
│   ├── collobartion
│   ├── _index.md
│   └── whiteboarding
│       ├── _index.md
│       └── video-1.md
├── getting-started
│   ├── _index.md
│   ├── onboarding
│   │   ├── designs.md
│   │   ├── _index.md
│   │   ├── kanvas.md
│   │   ├── profile copy.md
│   │   └── profile.md
│   └── tasks
│       ├── _index.md
│       ├── profile.md
│       └── video-1.md
├── _index.md
└── integrations
    ├── _index.md
    └── video-3.md

results ;
https://deploy-preview-511--bejewelled-pegasus-b0ce81.netlify.app/video-content/

@vishalvivekm vishalvivekm added the pr/do-not-merge PRs not ready to be merged label Apr 28, 2025
Signed-off-by: Vivek Vishal <[email protected]>
@vishalvivekm vishalvivekm requested a review from zihanKuang April 29, 2025 20:56
@vishalvivekm
Copy link
Contributor Author

vishalvivekm commented Apr 29, 2025

@zihanKuang
I'm requesting for your thoughts and suggestions on how should we organize our videos between categories and subcategories and sections, for example, should we have a basics videos category section?.

Right now, just for example, the rough structure that I have in this draft:
when you visit https://deploy-preview-511--bejewelled-pegasus-b0ce81.netlify.app/video-content/
you'll see the following categories and subcategories

  • features
    ---> collaboratoin
    --> whiteboarding

  • getting-started
    ---> onboarding
    ---> tasks

  • integrations
    ---> integration-1
    ---> integration-2

We definitely can take inspiration from a couple of places and I'm sharing some ((will keep adding)) as you're giving this a thought:

Copy link
Contributor

@zihanKuang zihanKuang left a comment

Choose a reason for hiding this comment

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

Hi @vishalvivekm ,here are a few suggestions I’d like to share:

1. High-level structure:

  • Getting Started (as a standalone tab for newcomers)
  • Cloud
  • Kanvas
  • Integrations

2. Subcategories (inside Cloud and Kanvas):
Since some users may want to dive deeper, we could consider dividing the content into:

  • Basic (for beginners)
  • Advanced (for more experienced users)

3. Keep it simple for now:
Since we don’t have a large number of videos yet, perhaps it's better to start with fewer categories. Videos are relatively “expensive” to produce and maintain, so keeping the structure simple in the beginning could help users focus and also be easier for us to manage. Once we have more content, we could always expand the structure later😎

4. Also, a small visual suggestion:
The current background feels a bit segmented and heavy, especially where it meets the black section. Maybe we could try using a gradient background instead?

image

These are just my thoughts — feel free to take or leave them! 😊

@vishalvivekm vishalvivekm changed the title [draft] video-landing-page feat: video-landing-page May 1, 2025
vishalvivekm and others added 3 commits May 1, 2025 21:21
Signed-off-by: GitHub <[email protected]>
Signed-off-by: vishalvivekm <[email protected]>
@vishalvivekm vishalvivekm merged commit a1f9145 into layer5io:master May 1, 2025
6 checks passed
@vishalvivekm
Copy link
Contributor Author

is to be followed up by #512..

@vishalvivekm
Copy link
Contributor Author

Thank you for your feedback @zihanKuang
I've created an issue tracking these.

Once, the contrib docs is ready on how to create videos, We can structure the videos just as you suggested.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants