Opened 13 months ago

Closed 13 months ago

Last modified 13 months ago

#3809 closed Bug (fixed)

Include encoding in .svg header

Reported by: vipsoft Owned by: Anthon Pang
Priority: normal Milestone: 1.12 - The Great 1.x Backlog
Component: UI - UX (AngularJS, twig, less) Keywords:
Cc: Sensitive: no

Description

The fix in [7873deb] doesn't entirely resolve the issue with .svg files not being served correctly.

On older Apache servers, there is no mime-type defined for svg+xml.

On Ubuntu 12.04, the Apache server's "magic" file expects the header to conform to:

0	string		\<?xml
>38	string		\<\!DOCTYPE\040svg	image/svg+xml

This ticket is to update all .svg files to have the following header:

<?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'>

Currently, the UserCountry .svg files have:

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

while logo.svg has no header.

Change History (3)

comment:1 Changed 13 months ago by vipsoft (robocoder)

  • Owner set to Anthon Pang
  • Resolution set to fixed
  • Status changed from new to closed

In 266da1c69318a4ce171bfc688b1bf39f328c2a87:

fixes #3809 - add encoding to .svg files

comment:3 Changed 13 months ago by matt (mattab)

I didn't notice it before, nice fix.

Note: See TracTickets for help on using tickets.