Skip to content

initial add faq section - #29

Closed
pspspspstty wants to merge 5 commits into
mainfrom
aeson/faq-section
Closed

initial add faq section#29
pspspspstty wants to merge 5 commits into
mainfrom
aeson/faq-section

Conversation

@pspspspstty

Copy link
Copy Markdown
Contributor

initial commit for faq section. i took the example FAQs from the project redesign meeting notes in notion, so if possible could we check to see if we want to change/add more?

i'm not sure if we want a tablet view? old project website has a tablet view but current figma uses a smartphone view instead. for the FAQ section it would pretty much be the same obviously but it might be a topic to bring up?

also might be good to note that i copied the max-width variable that the navbar uses for a small tablet so they would match. not sure if we will standardize that later in the redesign process

initial commit for faq section. i took the example FAQs from the project redesign meeting notes in notion, so if possible could we check to see if we want to change/add more?

i'm not sure if we want a tablet view? old project website has a tablet view but current figma uses a smartphone view instead. for the FAQ section it would pretty much be the same obviously but it might be a topic to bring up?

also might be good to note that i copied the max-width variable that the navbar uses for a small tablet so they would match. not sure if we will standardize that later in the redesign process
@vercel

vercel Bot commented Jul 6, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
projects-website Ready Ready Preview, Comment Jul 7, 2026 7:24am

Request Review

@NathanRLiu

Copy link
Copy Markdown

Is it necessary to use 'use client' for the FAQ?

@NathanRLiu

Copy link
Copy Markdown

Also, I don't think there was a design for the expanded FAQ item element so I'm going to ask design to get back to me on that. It looks pretty good for the most part but I think there could be more distinction between the question and the answer.

@NathanRLiu

Copy link
Copy Markdown

Also, when your mouse is hovered over one of the FAQ items and you try to scroll, it doesn't work. Can we fix this?

@pspspspstty

Copy link
Copy Markdown
Contributor Author

the redesign figma had a preview for it that i checked out yesterday. i don't know if they changed it since i last checked so i'll check it again tonight and tomorrow and make another commit to make it look more like the redesign.

@pspspspstty

Copy link
Copy Markdown
Contributor Author

i invoked 'use client' so i could use useState, but it looks like it's mentioned in page.tsx? so i got rid of it and it should still work. if you want i can instead try using

Details instead for each question but i think that doesn't allow for animations

@pspspspstty

Copy link
Copy Markdown
Contributor Author

whoops i didn't know it would do that. i mean that i could use the details HTML element instead

fix scrolling when mouse/finger is over the question box; remove redundant 'use client' directive
update formatting to more closely match what was in the notion
@NathanRLiu

Copy link
Copy Markdown

I think the question text should be more bold

NathanRLiu and others added 2 commits July 12, 2026 21:23
* added the timeline component

* adjusted some minor spacing

* changed timeline so that it auto shows the earliest upcoming event

---------

Co-authored-by: khanggn <khn042@ucsd.edu>
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