Skip to content

Add partial support for SVG - #73

Open
qarmin wants to merge 1 commit into
bojand:masterfrom
qarmin:svg
Open

Add partial support for SVG#73
qarmin wants to merge 1 commit into
bojand:masterfrom
qarmin:svg

Conversation

@qarmin

@qarmin qarmin commented Aug 20, 2022

Copy link
Copy Markdown
Contributor

Closes #60

This add partial support for SVG.
Most of SVG files which I checked started with <svg

<svg width="40" height="40" viewBox="0 0 40 40" fill="none" xmlns="http://www.w3.org/2000/svg">

but some starts with:

<?xml version="1.0" standalone="no"?><!DOCTYPE svg PUBLIC "-//W3C//DTD SVG 1.1//EN" "http://www.w3.org/Graphics/SVG/1.1/DTD/svg11.dtd"><svg t="1604130777424" class="icon"

which makes it impossible to simply search for an SVG file

@qarmin
qarmin force-pushed the svg branch 2 times, most recently from 1fa8410 to 014f9ec Compare August 20, 2022 06:45
@johansmitsnl

Copy link
Copy Markdown

Not all svg files start with just <svg. Here a header of a svg:

<?xml version="1.0" encoding="UTF-8"?>
<svg id="Layer_1" xmlns="http://www.w3.org/2000/svg" version="1.1" viewBox="0 0 850.39 850.39">
  <!-- Generator: Adobe Illustrator 29.2.1, SVG Export Plug-In . SVG Version: 2.1.0 Build 116)  -->
...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SVG finder

2 participants