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

feature/96/faq #123

Merged
merged 3 commits into from
Mar 14, 2024
Merged

feature/96/faq #123

merged 3 commits into from
Mar 14, 2024

Conversation

norachams
Copy link
Contributor

Screenshot 2024-03-03 at 2 42 44 AM Screenshot 2024-03-03 at 2 42 27 AM

@norachams norachams requested a review from aidantrabs as a code owner March 3, 2024 07:43
@norachams norachams linked an issue Mar 3, 2024 that may be closed by this pull request
src/components/index.tsx Outdated Show resolved Hide resolved
src/components/sections/FAQsection.tsx Outdated Show resolved Hide resolved
src/components/sections/FAQsection.tsx Outdated Show resolved Hide resolved
tailwind.config.mjs Outdated Show resolved Hide resolved
@teoh4770 teoh4770 assigned teoh4770 and norachams and unassigned teoh4770 Mar 3, 2024
@teoh4770 teoh4770 added the enhancement New feature or request label Mar 3, 2024
@juancwu
Copy link
Contributor

juancwu commented Mar 4, 2024

Are the comments marked as resolved because they are being worked on @norachams ? If yes, a comment saying they are being worked on would be nice if changes have not been committed.

onClick={() => toggleAccordion(index)}
>
<h6 className="text-black">{item.question}</h6>
<span>{activeIndex === index ? '^' : '⌄'}</span>
Copy link
Contributor

Choose a reason for hiding this comment

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

I think using an proper icon would make it look better. There is a slight change in position and size when active.

I recommend using:

A list of available icons:

Copy link
Contributor

Choose a reason for hiding this comment

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

@norachams Just making sure this doesn't get forgotten.

@juancwu juancwu merged commit a43b492 into main Mar 14, 2024
3 checks passed
@juancwu juancwu deleted the feature/96/faq branch March 14, 2024 01:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] Create FAQ section and styling
5 participants