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

Accordion not working #1

Open
LogikaServices7 opened this issue Jun 10, 2021 · 6 comments
Open

Accordion not working #1

LogikaServices7 opened this issue Jun 10, 2021 · 6 comments

Comments

@LogikaServices7
Copy link

Hello! I'm using your file for class and I've noticed the accordion isn't functioning properly, (not at all) the buttons are static. I was able the alter the size and actual button, but It won't reveal/unreveal the message inside the box... Please help! Thank you in advance!

@MuriungiPatrick
Copy link
Owner

First you need to link the bootstrap js to the theming kit.html

@fdes05
Copy link

fdes05 commented Jun 10, 2021

@LogikaServices7 Hi there, I ran into the same issue and just had to replace the two scripts at the very bottom of the 'theming-kit.html' file as the src (source) of those two scripts are looking for the bootstrap files on your local machine. It works after I replaced them with the script tags provided in the Bootsrap documentation at the following link:
https://getbootstrap.com/docs/5.0/getting-started/introduction/

I used the two 'separate' scripts as follows:

<script src="https://cdn.jsdelivr.net/npm/@popperjs/core@2.9.2/dist/umd/popper.min.js" integrity="sha384-IQsoLXl5PILFhosVNubq5LC7Qb9DXgDA9i+tQ8Zj3iwWAwPtgFTxbJ8NT4GN1R8p" crossorigin="anonymous"></script> <script src="https://cdn.jsdelivr.net/npm/bootstrap@5.0.1/dist/js/bootstrap.min.js" integrity="sha384-Atwg2Pkwv9vp0ygtn1JAojH0nYbwNJLPhwyoVbhoPwBhjQPR5VtM2+xf0Uwh9KtT" crossorigin="anonymous"></script>

hope that helps.

@LogikaServices7
Copy link
Author

Hey!!! I added the bootstrap cdn links and it works just fine. Thank you both for your help!

@iHamzaKhanzada
Copy link
Contributor

I made a PR for this issue #3

@iHamzaKhanzada
Copy link
Contributor

@MuriungiPatrick please close this issue as well as the PR has merged

@adriangonzalez
Copy link

adriangonzalez commented Jul 21, 2021

@iHamzaKhanzada It looks like the PR has been merged, but the commit isn't showing up in main.

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

No branches or pull requests

5 participants