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

SVG Icons Inline? #1492

Open
2 of 3 tasks
ithilgers opened this issue Jun 27, 2024 · 1 comment
Open
2 of 3 tasks

SVG Icons Inline? #1492

ithilgers opened this issue Jun 27, 2024 · 1 comment
Labels

Comments

@ithilgers
Copy link

Bug Report

Prerequisites

  • Can you reproduce the problem on TYPO3 v11.5 LTS
  • Can you reproduce the problem on TYPO3 v12.4 LTS
  • Did you perform a cursory search
    to see if your bug or enhancement is already reported?

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?

@ithilgers ithilgers added the bug label Jun 27, 2024
@stephankellermayr
Copy link
Contributor

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants