benjaminkott / bootstrap_package

Bootstrap Package delivers a full configured theme for TYPO3, based on the Bootstrap CSS Framework.
https://www.bootstrap-package.com/
MIT License
338 stars 205 forks source link

SVG Icons Inline? #1492

Open ithilgers opened 4 months ago

ithilgers commented 4 months ago

Bug Report

Prerequisites

Description

Meine HTML-Datei werden übertrieben groß, über 5 MB, aufgrund von Inline SVG-Dateien in der Navigation / Subnavigation.

Steps to Reproduce

  1. SVG-Datei in der Navigation / Subnavigation verwenden.
  2. Überprüfung der generierten HTML-Dateigröße.

Expected behavior

Kleinere HTML-Dateien

Actual behavior

Die HTML-Datei wird übertrieben groß, über 5 MB, aufgrund von eingebetteten SVG-Dateien.

Screenshots

image

Versions

TYPO3: 12.4.16 Bootstrap Package: 14.0.7

Anmerkung

Wenn ich in der Datei vendor/bk2k/bootstrap-package/Classes/Icons/FileIcon.php diese Abfrage auskommentiere, ist das Icon nicht mehr ein Inline-SVG und daher die HTML-Dateien kleiner.

if ($image->getExtension() === 'svg') {
    return SvgUtility::getInlineSvg('', $image, $width, $height);
}

Frage

Warum wurde sich entschieden die SVG Inline einzubinden?

stephankellermayr commented 4 months ago

Nur meine Meinung, aber beim Thema Icons hat man hier prinzipiell zu kurz gedacht und es "einfach passieren lassen".

Du könntest als Alternative EXT:iconpack in Kombination mit EXT:bootstrap_package_iconpack verwenden, und hättest dadurch eine höhere Flexibilität bei der Auswahl deiner Icon Sets und bei der Wahl der Ausgabe.