BugFix: missing exit wheel in piemenus of temperament, invert and some other blocks

[ksraj123]

Reference #2609, #2630, #2767

This PR does the following

• fixes the no exit wheel in piemenu bug for piemenus in invert, temperament and several other blocks
• fixes linting issues and prettifies code in js/piemenu.js

Details

Exit wheel in some piemenus of some blocks are missing even though there is space for it. Once the problematic piemenus are opened they could only be closed by clicking on the already selected item again which is not intuitive. This leads to bad user experience as and inconsistency in the user interface as the exit wheel is present in other piemenus.

Before

• Temperament

• Invert

Same issue is present in piemenus of a few other blocks as well.

After

• Temperament

• Invert

[meganindya]

Cool. Just add a .style.cursor = pointer for all the × buttons.

[ksraj123]

@meganindya Apparently this issue was present in almost all of of the pie menus across all blocks.

The blank space next to the x button has pointer but not the x button, ideally the blank space navItem should be disabled.

For exit wheels which had multiple clickable navItems like the number wheel, the selected navItem did not had the cursor even through for our use case the same could be clicked again. So ideally, all clickable navItems should have pointer as the cursor at all times.

These issues have been fixed.

[meganindya]

Yeah. These inconsistencies have existed all along. Somehow, haven't gotten along to fixing those,