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

Loading Bootstrap file from Option.html gives an error #555

Open
1 of 3 tasks
harshit-soora opened this issue Dec 16, 2020 · 0 comments · May be fixed by #556
Open
1 of 3 tasks

Loading Bootstrap file from Option.html gives an error #555

harshit-soora opened this issue Dec 16, 2020 · 0 comments · May be fixed by #556

Comments

@harshit-soora
Copy link

I'm submitting a ...

  • bug report
  • feature request
  • support request => Please do not submit support requests here, instead ask your query in our Gitter channel

Current behavior:

Whenever you click for the settings page option.html from the login page login.html, the background loads a javascript file bootstrap.min.js from the web, that in return leads to an error

Expected behavior:

There will not be any such error

Steps to reproduce:

Instead of calling the javascript file the web, we do have it inside our local folder /bootstrap
So we make a reference call to the same

Related code:

<script src="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.7/js/bootstrap.min.js" integrity="sha384-Tc5IQib027qvyjSMfHjOMaLkfuWVxZxUPnCJA7l2mCWNIpG9mGCD8wGNIcPD7Txa" crossorigin="anonymous"></script>

Inside option.html

Solution:

Simply we will call it as

<script src="bootstrap/bootstrap.min.js" integrity="sha384-Tc5IQib027qvyjSMfHjOMaLkfuWVxZxUPnCJA7l2mCWNIpG9mGCD8wGNIcPD7Txa" crossorigin="anonymous"></script>

Screenshots of the issue:

error

Ubuntu 20.04

Would you like to work on the issue?

Yes, sure!
@harshit-soora harshit-soora linked a pull request Dec 16, 2020 that will close this issue
6 tasks
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 a pull request may close this issue.

1 participant