-
Notifications
You must be signed in to change notification settings - Fork 2
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/organizer carousel #535
base: main
Are you sure you want to change the base?
Conversation
@@ -1,4 +1,4 @@ | |||
import { Landing, ChooseCharacter, FAQ, Sponsors, Partners } from "./sections"; | |||
import { Landing, ChooseCharacter, FAQ, Sponsors, Partners, Organizers } from "./sections"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[prettier] reported by reviewdog 🐶
import { Landing, ChooseCharacter, FAQ, Sponsors, Partners, Organizers } from "./sections"; | |
import { | |
Landing, | |
ChooseCharacter, | |
FAQ, | |
Sponsors, | |
Partners, | |
Organizers, | |
} from "./sections"; |
"use client"; | ||
|
||
import React, { useEffect, useState } from "react"; | ||
import Image from "next/image"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'Image' is defined but never used.
|
||
import React, { useEffect, useState } from "react"; | ||
import Image from "next/image"; | ||
import box from "@/assets/images/center_chat_box.svg"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'box' is defined but never used.
import React, { useEffect, useState } from "react"; | ||
import Image from "next/image"; | ||
import box from "@/assets/images/center_chat_box.svg"; | ||
import boxBG from "@/assets/images/center_chat_box_bg.svg"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'boxBG' is defined but never used.
import Image from "next/image"; | ||
import box from "@/assets/images/center_chat_box.svg"; | ||
import boxBG from "@/assets/images/center_chat_box_bg.svg"; | ||
import Link from "next/link"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'Link' is defined but never used.
|
||
export const InfiniteMovingCards = ({ | ||
items, | ||
direction = "left", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'direction' is assigned a value but never used.
export const InfiniteMovingCards = ({ | ||
items, | ||
direction = "left", | ||
speed = "fast", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'speed' is assigned a value but never used.
items, | ||
direction = "left", | ||
speed = "fast", | ||
pauseOnHover = true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'pauseOnHover' is assigned a value but never used.
}) => { | ||
const containerRef = React.useRef<HTMLDivElement>(null); | ||
const scrollerRef = React.useRef<HTMLUListElement>(null); | ||
const [start, setStart] = useState(false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'start' is assigned a value but never used.
scrollerRef.current.lastChild && | ||
scrollerRef.current.removeChild(scrollerRef.current.lastChild); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Expected an assignment or function call and instead saw an expression.
Deploy preview for irvinehacks-site-2025 ready!
|
Regarding your comment about needing to add |
Fun Organizer Carousel!
I put all the images and names and departments into sanity so I hope it works. I added localhost:3000 to the CORS origins in the sanity dashboard because i was running into CORS issues. LMK if there are better solutions.
Also I added Images to be optimized by next in the next.config. Only from Sanity.
Pictures that aren't added (as in if they weren't in the Zothacks headshots) are replaced with a placeholder: volunteer_sprite.png. Maybe we can pick a different one.