Skip to content

Commit 8b339b7

Browse files
authored
fix: Fix include of bootstrap-icons (ietf-tools#4039)
This got broken in ietf-tools#4035, which included some things it wasn't meant to.
1 parent 57aee14 commit 8b339b7

2 files changed

Lines changed: 5 additions & 3 deletions

File tree

ietf/static/css/ietf.scss

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,11 @@ $popover-max-width: 100%;
5656
// Utilities
5757
@import "bootstrap/scss/utilities/api";
5858

59+
// Make the bootstrap icons available.
60+
$bootstrap-icons-font-src: url("npm:bootstrap-icons/font/fonts/bootstrap-icons.woff2") format("woff2"),
61+
url("npm:bootstrap-icons/font/fonts/bootstrap-icons.woff") format("woff");
62+
@import "bootstrap-icons/font/bootstrap-icons";
63+
5964
// Leave room for fixed-top navbar...
6065
body {
6166
padding-top: 60px;

ietf/static/js/ietf.js

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,9 +16,6 @@ import "bootstrap/js/dist/tooltip";
1616

1717
import jquery from "jquery";
1818

19-
// Make the bootstrap icons available. Yes, we require CSS in JS, parcel wants this.
20-
require("bootstrap-icons/font/bootstrap-icons.css");
21-
2219
window.$ = window.jQuery = jquery;
2320
if (!process.env.BUILD_DEPLOY) {
2421
// get warnings for using deprecated jquery features

0 commit comments

Comments
 (0)