Remove aria-labelledby and ids from SVGs

[ericcornelissen]

Issue: #991

Description

This Pull Request removes the aria-labelledby attribute from all SVGs and the id attribute from all SVG <title>s. I updated the SVGLint configuration and documentation (i.e. README.md and CONTRIBUTING.md) accordingly.

As mentioned in the issue I tested this on Windows 10 using: Windows 10 build in Narrator, NVAccess, ChromeVox. Additionally we might want to test on MacOS, smartphone OSes and Linux (e.g. Ubuntu)(?).

Another thing we may consider doing is informing (existing) contributors about this change, as they will probably still include the two attributes when they submit a new icon 🤔

[birjj]

👍 Great! Thanks for putting in the work to test and update this!

As for current contributors, I'm fine with informing them if they submit a PR with the old format - the build step should fail in those cases regardless.

[ericcornelissen]

As for current contributors, I'm fine with informing them if they submit a PR with the old format - the build step should fail in those cases regardless.

I agree. Just figured a heads-up in the template might be an idea. I also don't think adding temporary messages to the template is a particularly good idea.