Problem/Motivation

According to MDN document, SVG requires xmlns attribute in outermost svg element. But current throbber SVG images has no xmlns attribute.

Steps to reproduce

Access to /core/misc/feed.svg using a web browser, this svg file is recognized as a SVG image. But access to /core/misc/throbber-active.svg, this svg file is recognized as a normal XML document.

The feed.svg is shown as an image file.

The throbber-inactive.svg is shown as a xml file.

Proposed resolution

Add xmlns attribute in the root svg element.

Remaining tasks

User interface changes

No.

API changes

No.

Data model changes

No.

CommentFileSizeAuthor
feeds-svg.png71.57 KBtom konda
throbber-inactive-svg.png107.18 KBtom konda

Issue fork drupal-3466480

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

Tom Konda created an issue. See original summary.

tom konda’s picture

tom konda
Primary language Japanese
Location Kanagawa, Japan
commented
Status: Active » Needs work

Tom Konda opened merge request !9110

tom konda’s picture

tom konda
Primary language Japanese
Location Kanagawa, Japan
commented
Status: Needs work » Needs review
smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community

Seems straight forward.

nod_ closed merge request !9110

  • nod_ committed 08f436ed on 11.x 
    Issue #3466480 by Tom Konda, smustgrave: New throbber SVG images lack...
nod_’s picture

nod_
Primary language French
Location Lille
commented
Title: New throbber SVG images are lack of xmlns attribute » New throbber SVG images lack xmlns attribute
Status: Reviewed & tested by the community » Fixed

Committed 08f436e and pushed to 11.x. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.