Faq eniola ademola

[EniolaAdemola]

Describe your change:

• [x] Add a Project?
• [ ] Fix a bug or typo in an existing Projects?
• [ ] Documentation change?

Checklist:

• [ ] I have read CONTRIBUTING.md.
• [ ] This pull request is all my own work -- I have not plagiarized.
• [ ] I know that pull requests will not be merged if they fail the automated tests.
• [ ] This PR only changes one Project folder. To ease review, please open separate PRs for separate algorithms.
• [ ] All new Project files are placed inside an existing directory.
• [ ] If this pull request resolves one or more open issues then the description above includes the issue number(s) with a closing keyword: "Fixes #ISSUE-NUMBER".
• [ ] My code follows the style guidelines of this project
• [ ] I have performed a self-review of my own code
• [ ] I have commented my code, particularly in hard-to-understand areas
• [ ] My changes generate no new warnings
• [ ] The title of my pull request is a short description of the requested changes.
• [ ] I have made corresponding changes to the documentation
• [ ] My PR needs documentation change

ATTACH SCREEN-SHOTS

Before	After
< Before Image >	< After Image >

[netlify[bot]]

✅ Deploy Preview for frontend-database ready!

Name	Link
🔨 Latest commit	e1f9ccdd0b404ae292ec2ef1bacb8c84fad960f1
🔍 Latest deploy log	https://app.netlify.com/sites/frontend-database/deploys/6521530cd802be00085a8220
😎 Deploy Preview	https://deploy-preview-17--frontend-database.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[pr-explainer-bot[bot]]

Pull Request Report

Hey there! I've created a report for the pull request. Let's dive in!

Changes

1. Modified ProjectList.json:

   • Added a new project "Animated Navigation" by Eniola Ademola.
   • Updated the tags for the "Payment Gateway System" project.

2. Added new files:

   • index.html for the FAQ Collapse project.
   • style.css for the FAQ Collapse project.
   • script.js for the FAQ Collapse project.

Suggestions

I have a few suggestions to improve the code:

1. In index.html for the FAQ Collapse project, consider adding a DOCTYPE declaration at the beginning of the file.

2. In index.html for the FAQ Collapse project, you can remove the integrity and crossorigin attributes from the <link> tag for the font-awesome stylesheet. These attributes are not necessary for local files.

3. In style.css for the FAQ Collapse project, you can remove the @import rule for the Google Fonts stylesheet. Instead, you can directly link the stylesheet in the HTML file.

Bugs

I couldn't find any potential bugs in the code. Great job!

Improvements

I found a couple of places in the code that could be refactored for better readability:

1. In script.js for the FAQ Collapse project, you can use arrow function syntax for the event listener callback function. Here's the updated code snippet:

   const toggles = document.querySelectorAll(".faq-toggle");
   
   toggles.forEach((toggle) => {
    toggle.addEventListener("click", () => {
      toggle.parentNode.classList.toggle("active");
    });
   });

2. In style.css for the FAQ Collapse project, you can remove the duplicated border-box property from the universal selector (*). It's already defined in the body selector.

Rating

I would rate the code a 7 out of 10 based on the following criteria:

• Readability: The code is generally easy to read and understand, but there are a few areas where it could be improved.
• Performance: The code seems to be efficient and doesn't have any obvious performance issues.
• Security: The code doesn't contain any security vulnerabilities that I could identify.

That's it for the pull request report! Let me know if you need any further assistance. Keep up the good work! 👍