rainstormstudio / nerd-icons.el

GNU General Public License v3.0
206 stars 22 forks source link

data: Update mdi icon references to newer code points #21

Closed mohkale closed 1 year ago

mohkale commented 1 year ago

CLOSES #20

nerd-fonts deprecated these legacy code points in v2.3.3 because they conflicted with non-Latin letters and then removed them in v3.0.0.

I updated the code-points by using expanding them into unicode, using nerdfix to pick the recommended replacement and then converting them back to escape sequences. I did this for both md and mdi but only mdi had actually changed so the diff shows it and no changes for md.

I took a look at a couple of the new mdi ones and looks like the existing font installed by nerd-icons.el already supports the new codepoints so nothing else should need to be updated.

Note: The following icons look to have been removed upstream and are have been removed by this PR as well:

seagle0128 commented 1 year ago

It looks good for me! Let @rainstormstudio make the final call.