When changes from the default icon to the other, it is very difficult.
As previously described in
✨
Add template for the icon
Postponed: needs info
, need to rewrite whole Drupal.extlink.applyClassAndSpan to customize icon HTML output on current implementation.
Implement icon HTML output using Drupal.theme() JavaScript functions.
If defines icon HTML output on Drupal.theme(), it is easier to override HTML output from an other theme, no need to rewrite whole Drupal.extlink.applyClassAndSpan().
Needs work
2.0
Code
The patch will have to be re-rolled with new suggestions/changes described in the comments in the issue.
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
Can this be turned to MR,
May be a good scenario to add test coverage for
Example test module that overrides this setting
Test case verifying that.
Will have to also take into account that jquery and tel: has been added too