Fixes 2814, 2815

[simondate]

Fixes -

• WYSIWYG editor stripping out html #2814
• CKEditor toolbar element 'RemoveFormat' not loaded in #2815

Two issues with CKEditor.

[simondate]

Just updated this PR as it was still stripping the aria-hidden="true" attribute.

Note for testers - You'll need to update the conf/config.json file, to include this -

 "ckEditorHtmlSupport": {
    "allow": [
      {
        "name": "^(blockquote|ul|ol|span)$",
        "attributes": true,
        "classes": true,
        "styles": true
      },
      {
        "name": ".*",
        "classes": true,
        "styles": true
      },
      {
        "name": "^a$",
        "attributes": {
          "target": true
        }
      }
    ],
    "allowEmpty": [
      "span", 
      "i"
    ]
  },

[StuartNicholls]

Hey @simondate, I'm curious as to why we're allowing the tag here?

[simondate]

What do you mean? Do you think we should also expose options here in config/config.json?

I guess that will make this release slightly more future proof? As if in the future more attributes are added we can extend it further there.

[StuartNicholls]

Sorry, that was supposed to say, <i> tag, rather than making the rest of the line italicised!

So was just asking why HTML italic tag is there.

[simondate]

This was just in the example that I copied from CKEditior's website. I know that some other icon packs such as Font Awesome use <i>, so perhaps someone else may find it useful.

[simondate]

Just discovered that the any links to Glossary objects also weren't working.

The latest commit fixes this too.