fix: normalize renderIcon types

[adamalston]

Closes #16735

Normalized renderIcon types.

Changelog

Changed

• Normalized renderIcon types.

Testing / Reviewing

Paste the code from the issue at the bottom of Tabs.tsx and check for errors.

[netlify]

✅ Deploy Preview for v11-carbon-web-components ready!

Name	Link
🔨 Latest commit	1fe9d8d
🔍 Latest deploy log	https://app.netlify.com/sites/v11-carbon-web-components/deploys/67bdd1e628974f0008204b6e
😎 Deploy Preview	https://deploy-preview-18683--v11-carbon-web-components.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[netlify]

✅ Deploy Preview for carbon-elements ready!

Name	Link
🔨 Latest commit	1fe9d8d
🔍 Latest deploy log	https://app.netlify.com/sites/carbon-elements/deploys/67bdd1e6785f1200083f2c16
😎 Deploy Preview	https://deploy-preview-18683--carbon-elements.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[netlify]

✅ Deploy Preview for v11-carbon-react ready!

Built without sensitive environment variables

Name	Link
🔨 Latest commit	1fe9d8d
🔍 Latest deploy log	https://app.netlify.com/sites/v11-carbon-react/deploys/67bdd1e6aa39a50008f7cae5
😎 Deploy Preview	https://deploy-preview-18683--v11-carbon-react.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.34%. Comparing base (faf6498) to head (1fe9d8d).
Report is 3 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #18683   +/-   ##
=======================================
  Coverage   84.34%   84.34%           
=======================================
  Files         409      409           
  Lines       14651    14651           
  Branches     4824     4774   -50     
=======================================
  Hits        12358    12358           
- Misses       2131     2132    +1     
+ Partials      162      161    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[adamalston]

I think all JSDoc comments for renderIcon props should be standardized to use the same text. The props themselves will be consistent with the changes in this pull request, so I think the JSDoc comments for them should be consistent. Curious to hear whether others agree or have any other thoughts.

[Gururajj77]

Hey

I think all JSDoc comments for renderIcon props should be standardized to use the same text. The props themselves will be consistent with the changes in this pull request, so I think the JSDoc comments for them should be consistent. Curious to hear whether others agree or have any other thoughts.

Hey @adamalston , I agree with your suggestion. Can you make change to the comments, and we can see what we can do with that.

[adamalston]

I have updated all JSDoc comments. There are a few different ways I envision things could go now:

• You all think the updates are better. In this case, I will keep all changes.
• You all think the updates make things worse. In this case, I will revert all changes (i.e., 1fe9d8d).
• You all think some of the updates make things better and others make things worse. In this case, I will revert everything you think is worse.

Obviously, I'll do whatever you guys want. My goal here was to try and introduce some consistency.