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

Responsive FAQ Section #95

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

eshitatalukdar08
Copy link

Issue number #88
I have added a FAQ section including a few questions. The features include:

  1. Elegant Design:
    Utilized a black and gold color scheme to align with the website's theme.
    Added subtle box shadows and hover effects for a polished look.

  2. Responsive FAQ Section:
    Questions are clickable, expanding to reveal answers with a smooth animation.
    The layout adjusts gracefully across various screen sizes, ensuring optimal usability on desktop and mobile devices.

  3. "Couldn't Find Your Answer?" Section:
    It includes a prompt with a "Contact Us" link styled for visibility and elegance.
    The link redirects users to the contact page (contact.html).

  4. Interactive Experience:
    JavaScript toggles the visibility of answers, providing an intuitive user experience.
    Smooth transitions for hover effects on the "Contact Us" link, with font size and color changes.

Thank you for assigning this task to me, I hope I was able to contribute.

Copy link

vercel bot commented Jan 9, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
603-work-ved ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 10, 2025 7:17am

@swaparup36
Copy link
Collaborator

First of all good job @eshitatalukdar08. It looks good but, can you please move the code from faq.css file to the /assets/css/style.css and faq.js to /assets/js/script.js. It's important to maintain a good folder structure and make it understandable

@eshitatalukdar08
Copy link
Author

Thank you so much for the feedback! I'll go ahead and move the code from faq.css to /assets/css/style.css and faq.js to /assets/js/script.js. I appreciate you pointing this out!

@eshitatalukdar08
Copy link
Author

@swaparup36 Do I need to create a new PR for that?

@Mayank77maruti
Copy link
Owner

Mayank77maruti commented Jan 11, 2025

@swaparup36 Do I need to create a new PR for that?

@eshitatalukdar08 If this pr is in mess then you can create a new pr , no worries.

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.

3 participants