Skip to content

Commit db24afe

Browse files
authored
Merge pull request #1802 from IrAlfred/fix-bootstrap-icons-font-paths-production
fix(other): fix Bootstrap Icons not visible in production mode (DEBUG=false)
2 parents be4a5b2 + e1d43ae commit db24afe

File tree

2 files changed

+13
-2
lines changed

2 files changed

+13
-2
lines changed

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
/site.js
33
/site.css
44
/site/
5+
/fonts/
56
/todo
67
/tests/selenium/creds.py
78
/tests/selenium/local_creds.py

scripts/config_gen.php

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -318,14 +318,24 @@ function write_config_file($settings, $filters) {
318318
* @return void
319319
*/
320320
function append_bootstrap_icons_files() {
321+
// Ensure target font directories exist for both deployment modes:
322+
// - Running from site/: fonts are expected under site/fonts relative to site/site.css
323+
// - Running from the main directory: fonts are expected under fonts relative to site.css
321324
if (!is_dir("site/fonts")) {
322325
mkdir('site/fonts', 0755);
323326
}
327+
if (!is_dir("fonts")) {
328+
mkdir('fonts', 0755);
329+
}
324330
$source_folder = VENDOR_PATH.'twbs/bootstrap-icons/font/fonts/';
325331
$files = glob("$source_folder*.*");
326332
foreach($files as $file){
327-
$dest_forlder = str_replace($source_folder, "site/fonts/", $file);
328-
copy($file, $dest_forlder);
333+
// Copy for site/ deployments (CSS loaded from site/site.css → ./fonts resolves to site/fonts)
334+
$dest_folder_site = str_replace($source_folder, "site/fonts/", $file);
335+
copy($file, $dest_folder_site);
336+
// Copy for main-directory deployments (CSS loaded from site.css → ./fonts resolves to fonts)
337+
$dest_folder_root = str_replace($source_folder, "fonts/", $file);
338+
copy($file, $dest_folder_root);
329339
}
330340
}
331341

0 commit comments

Comments
 (0)