replacing the icons

[brianteeman]

Unfortunately it is not as simple as replacing the content attribute with the CSS variables currently being used as the built-in css also manipulates the icons eg

https://github.com/ranbuch/accessibility/blob/6077514135979f53018e696e8e27e51b0b5305c3/src/main.ts#L652-L657

[ranbuch]

@brianteeman Version 5.0.4 will support more CSS variables in order to adjust the icons better. Trust me, you don't want to write your own CSS. Even if you'll manage to do so your code might break in future releases.

Keep me posted.

[brianteeman]

Switching from material icons to my own icon set lets me see various other places where content variables are needed. I will try to list them all here

  text: "".concat(!this.options.icon.useEmojis ? 'close' : 'X')
  text: "".concat(!this.options.icon.useEmojis ? 'refresh' : '♲')
  content: ${!this._acc.options.icon.useEmojis ? '"mic"' : '"🎤"'}

and finally 'class': this._acc.options.icon.useEmojis ? '' : 'material-icons',
which I guess really should be using the variable fontClass

[brianteeman]

i am guessing that similar changes are also required for the main icon as that is also hardcoded for either material or emoji

[ranbuch]

@brianteeman Please try version 5.1.4 and let me know.

[brianteeman]

unless I am missing something this doesnt work. With this PR the code generated is <button class="_menu-close-btn _menu-btn iconfont" title="Hotkey: ctrl+alt+a" tabindex="0">close</button>

the problem is that while material will replace the close with an icon fotnaweseome and others will not as they rely on a class

[ranbuch]

@brianteeman The readme file has this example (I should add it to the site as well):

const options = {
    icon: {
        fontFaceSrc: ['https://cdnjs.cloudflare.com/ajax/libs/font-awesome/6.5.1/css/v4-font-face.min.css'],
        fontFamily: '"FontAwesome"',
        img: '[optional - URL for an image that will replace the menu icon]',
        closeIcon: '[optional - replacement text for the close menu icon]',
        resetIcon: '[optional - replacement text for the reset all icon]'
    }
};
new Accessibility(options);

closeIcon and resetIcon has been added according to your previous comments.

[brianteeman]

Yes I saw that but closeIcon/resetIcon does not work with fontawesome. did you try it?

[ranbuch]

I see now that it's only working for Emoji mode. Please try version 5.1.5.

[brianteeman]

that doesnt work either - did you test it yourself? the problem is the same as I already said

the problem is that while material will replace the close with an icon fontaweseome and others will not as they rely on a class

[ranbuch]

Please try version 6.0.1 and see updated documentation. It's a bit more complicated because we need to change the HTML itself sometimes, not just CSS variables.

[brianteeman]

thanks - will test later today and report back but at a quick glance it does appear to be on the right path

[kvonspiczak]

I'm trying to replace the icons with a local fontface, since we already have material icons loaded anyways. It seems to be working fine, however we're still getting an error in the developer console regarding a CSP violation:

Refused to load the stylesheet 'https://fonts.googleapis.com/icon?family=Material+Icons' because it violates the following Content Security Policy directive: "style-src-elem 'self' 'unsafe-inline' 'report-sample'".

Am I missing something here, or why is the tool still trying to load the icons from google? I did it like in the documentation:

const options = {
    icon: {
        fontFaceSrc: ['https://my-domain.com/_assets/some-random-hash/StyleSheets/main.min.css']
    }
};

[ranbuch]

Hi @kvonspiczak Your options object looks good. How / where can I reproduce the issue? Are you sure the injection is related to this library?