Skip to content

Add Explore page with video thumbnails and modal, use Header componen…#2

Open
mdejesus23 wants to merge 3 commits intostagingfrom
homepage-ul-grid-update
Open

Add Explore page with video thumbnails and modal, use Header componen…#2
mdejesus23 wants to merge 3 commits intostagingfrom
homepage-ul-grid-update

Conversation

@mdejesus23
Copy link
Collaborator

This PR added the Explore page, displaying a random selection of 30 video thumbnails via getCollectionDocs logic. It also implements the following changes:

  • Uses the existing Header component for navigation (I just used the existing logo).
  • Adds hover functionality to play videos without sound.
  • Opens videos in a modal with sound on click (using the existing modal component).
  • Updates the homepage to become the Explore page.
  • Sets the "Try Now" button to navigate to the Create page with react-router.
  • Uses 9:16 aspect ratio for thumbnails for a better visual experience.

@mdejesus23 mdejesus23 requested a review from jdwyer6 March 19, 2025 06:50
@netlify
Copy link

netlify bot commented Mar 19, 2025

Deploy Preview for micebandstaging ready!

Name Link
🔨 Latest commit d4a5754
🔍 Latest deploy log https://app.netlify.com/sites/micebandstaging/deploys/67db787a01c7330008d734ed
😎 Deploy Preview https://deploy-preview-2--micebandstaging.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.

--tw-numeric-figure: ;
--tw-numeric-spacing: ;
--tw-numeric-fraction: ;
--tw-ring-inset: ;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Looks like there was some auto formatting here or maybe it changed in pre-compilation. Can you git patch add changes (or possibly git revert and restore easier?) so that only your changes are in the PR so it's easier to review?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Sure, I understand. I'll make sure only my changes are reflected in the PR. I'll check if a git revert or a selective git add -p would be the best approach to clean it up. Thanks for the feedback!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

done reverting this file.

import { initializeApp } from "firebase/app";
import { getAuth, signInWithEmailAndPassword, signOut, onAuthStateChanged } from "firebase/auth";
import { getStorage, ref, uploadBytes, getDownloadURL, listAll, deleteObject } from "firebase/storage";
import { getFirestore, collection, addDoc, getDocs, updateDoc, doc, deleteDoc, setDoc, getDoc } from "firebase/firestore";
Copy link
Collaborator

Choose a reason for hiding this comment

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

same kind of thing here. Anywhere it's just format differences but not lines you added can we revert that? Sorry I know it's probably an auto formatter.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

done reverting this file

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