flameshot-org/flameshot

SVG optimization made some icons unreadable

Open

#3247 aperta il 28 giu 2023

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)C++ (1916 fork)batch import
BugHelp WantedVisuals

Metriche repository

Star
 (29.897 star)
Metriche merge PR
 (Merge medio 3g 8h) (9 PR mergiate in 30 g)

Descrizione

While trying to use some of the SVGs created for buttons on the webpage, I realized the img/buttonIconsBlack/square.svg with following code does not render in Firefox:

https://github.com/flameshot-org/flameshot/blob/236534b17b0318b05b4e28bb906c38c78bae84de/data/img/material/black/square.svg#L1C1-L3C7 :

<svg viewBox="0 0 24 24">
    <path d="M3,3V21H21V3"/>
</svg>

But the same file before the #281 (commit d09580b) with the following content work just fine:

<?xml version="1.0" encoding="UTF-8"?><!DOCTYPE svg PUBLIC "-//W3C//DTD SVG 1.1//EN" "http://www.w3.org/Graphics/SVG/1.1/DTD/svg11.dtd"><svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" version="1.1" width="24" height="24" viewBox="0 0 24 24"><path d="M3,3V21H21V3" /></svg>

The working icon: , the broken icon:

There are two SVG files in the line above but one of them is rendered in the browser. View page source to confirm.

Guida contributor

Direzione di ricerca
Il problema identifica che il file SVG 'data/img/material/black/square.svg' manca degli attributi XML richiesti (xmlns, width, height) a causa dell'ottimizzazione nel commit d09580b. Per risolvere, aggiungere gli attributi mancanti: xmlns="http://www.w3.org/2000/svg" width="24" height="24". Controllare anche altri SVG nella stessa directory che potrebbero essere stati interessati dalla stessa ottimizzazione e applicare correzioni simili se necessario. Non è richiesta una risposta dal manutentore poiché il problema e la soluzione sono chiaramente descritti.
Tech stack
Nessuno
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
SVGGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90