flameshot-org/flameshot

SVG optimization made some icons unreadable

Open

#3 247 ouverte le 28 juin 2023

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)C++ (1 916 forks)batch import
BugHelp WantedVisuals

Métriques du dépôt

Stars
 (29 897 stars)
Métriques de merge PR
 (Merge moyen 3j 8h) (9 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Le problème identifie que le fichier SVG 'data/img/material/black/square.svg' manque des attributs XML requis (xmlns, width, height) en raison de l'optimisation dans le commit d09580b. Pour corriger, ajoutez les attributs manquants : xmlns="http://www.w3.org/2000/svg" width="24" height="24". Vérifiez également les autres SVG dans le même répertoire qui pourraient avoir été affectés par la même optimisation et appliquez des corrections similaires si nécessaire. Aucune réponse du mainteneur n'est requise car le problème et la solution sont clairement décrits.
Stack technique
Aucun
Domaine
frontend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
SVGGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90