Remove obsolete destructuring in decorate.js::decorateIcons

[flx-sta]

remove obsolete destructuring in decorate.js::decorateIcons

Description

The destructuring here is unnecessary cause a NodeList, which is the return type for querySelectorAll supports forEach too and we also don't have to get rid of any references.

Related Issue

#40

Motivation and Context

is eliminates obsolete code

How Has This Been Tested?

By removing it in local development and everything still works fine

Screenshots (if appropriate):

n/a

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• I have signed the Adobe Open Source CLA.
• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.

[flx-sta]

push