Put the aria-label on the link and not the item wrapper

Note: not having the dev branch 8.x-2.x-dev published is not helping for the fork/MR workflow to work simply.

Note 2: Ok, I just realize now that the 8.x-2.x branch does not exist at all in the main repository.

MR https://git.drupalcode.org/project/fontawesome_menu_icons/-/merge_reques... created

Hello,
I've installed the module version 3.0.0 in Drupal 10.3.4 and added icons in the main navigation menu links followed by configuring the appearance to without text. Found the aria-label is being set in the <li> hence when the screen reader reads the menu link, it fails to identify the aria-label. After applying the MR!2, the aria-label is set in the <a> which is identified by the screen reader correctly. The issue is raised for 8.x-2.0, but it still persists in 3.0.0 and the provided MR applies in the latest version cleanly therefore need to port the changes in 3.x-dev. The MR resolves the issue therefore moving to RTBC, including before and after ss.
Thanks