flameshot-org/flameshot

SVG optimization made some icons unreadable

Aberta

#3.247 aberto em 28 de jun. de 2023

Ver no GitHubGuia do colaborador
 (3 comentários) (0 reação) (0 responsável)C++ (1.916 forks)batch import
BugHelp WantedVisuals

Métricas do repositório

Stars
 (29.897 estrelas)
Métricas de merge de PR
 (Mesclagem média 2d 23h) (12 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
O problema identifica que o arquivo SVG 'data/img/material/black/square.svg' carece dos atributos XML necessários (xmlns, width, height) devido à otimização no commit d09580b. Para corrigir, adicione os atributos ausentes: xmlns="http://www.w3.org/2000/svg" width="24" height="24". Verifique também outros SVGs no mesmo diretório que possam ter sido afetados pela mesma otimização e aplique correções semelhantes, se necessário. Nenhuma resposta do mantenedor é necessária, pois o problema e a solução estão claramente descritos.
Pilha de tecnologia
None
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
SVGGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
90