UI improvements hover tooltip

[cphelefu]

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes #2141
Fixes #2151
Fixes #2152
Fixes #2153

Type of change

• 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 not work as expected)
• This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.

https://cphelefu.github.io/geoview/

Checklist:

• I have build (rush build) and deploy (rush host) my PR
• I have connected the issues(s) to this PR
• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• My changes generate no new warnings
• I have created new issue(s) related to the outcome of this PR is needed
• I have made corresponding changes to the documentation
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes

---

This change is 

[kaminderpal]

Reviewable status: 0 of 18 files reviewed, 2 unresolved discussions (waiting on @cphelefu)

---

packages/geoview-core/src/core/components/common/layer-icon.tsx line 26 at r1 (raw file):

  const iconData = useIconLayerSet(layerPath);

  const iconImage: string = iconData && iconData.length > 0 ? iconData[0] : '';

iconData && iconData.length can be written as iconData?.length same what we have for variable numOfIcons

---

packages/geoview-core/src/core/components/hover-tooltip/hover-tooltip.tsx line 14 at r1 (raw file):

 * @returns {JSX.Element} the hover tooltip component
 */
export function HoverTooltip(): JSX.Element {

why we remove return type.

[cphelefu]

Reviewable status: 0 of 22 files reviewed, 2 unresolved discussions (waiting on @kaminderpal)

---

packages/geoview-core/src/core/components/common/layer-icon.tsx line 26 at r1 (raw file):

Previously, kaminderpal (Kamy) wrote…

iconData && iconData.length can be written as iconData?.length same what we have for variable numOfIcons

Done.

---

packages/geoview-core/src/core/components/hover-tooltip/hover-tooltip.tsx line 14 at r1 (raw file):

Previously, kaminderpal (Kamy) wrote…

why we remove return type.

Done.

[kaminderpal]

Reviewed 5 of 18 files at r1, 14 of 17 files at r2, 2 of 3 files at r3, 1 of 1 files at r4, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @cphelefu)

[jolevesq]

Reviewed 5 of 18 files at r1, 14 of 17 files at r2, 2 of 3 files at r3, 1 of 1 files at r4, 1 of 1 files at r5, 2 of 2 files at r6, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @cphelefu)